public inbox for gcc-prs@sourceware.org
help / color / mirror / Atom feed
From: jbach1@caramail.com
To: gcc-gnats@gcc.gnu.org
Subject: preprocessor/5520: There's a warning I miss with "if{} else{}" statement
Date: Mon, 28 Jan 2002 15:46:00 -0000	[thread overview]
Message-ID: <20020128234120.3185.qmail@sources.redhat.com> (raw)


>Number:         5520
>Category:       preprocessor
>Synopsis:       There's a warning I miss with "if{} else{}" statement
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    unassigned
>State:          open
>Class:          change-request
>Submitter-Id:   net
>Arrival-Date:   Mon Jan 28 15:46:01 PST 2002
>Closed-Date:
>Last-Modified:
>Originator:     Pierre Chatelier
>Release:        gcc 3.0.3
>Organization:
>Environment:
DOS (DJGPP)
>Description:
I think a warning shoul be raised when someone writes
if (condition);
{
}

instead of
if (condition)
{
}

(===== Watch at the ";" ====)

Actually, it can only be a mistake, and that's a pity it's not detected. It drives to severe bugs.

>How-To-Repeat:
int x=1;
if (x == 0);
{
   cout << "x equals zero" << endl;
}
>Fix:
int x=1;
if (x == 0)
{
   cout << "x equals zero" << endl;
}
>Release-Note:
>Audit-Trail:
>Unformatted:


             reply	other threads:[~2002-01-28 23:46 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-01-28 15:46 jbach1 [this message]
2002-01-28 18:16 Carlo Wood

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20020128234120.3185.qmail@sources.redhat.com \
    --to=jbach1@caramail.com \
    --cc=gcc-gnats@gcc.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).