mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Jesse Pollard <pollard@tomcat.admin.navo.hpc.mil>
To: andrea@suse.de, Ulrich Drepper <drepper@cygnus.com>
Cc: "linux-kernel@vger.kernel.or" <linux-kernel@vger.kernel.org>
Subject: Re: 2.2.18 signal.h
Date: Fri, 15 Dec 2000 15:56:52 -0600 (CST)	[thread overview]
Message-ID: <200012152156.PAA137696@tomcat.admin.navo.hpc.mil> (raw)

---------  Received message begins Here  ---------

> From: Andrea Arcangeli <andrea@suse.de>
> On Fri, Dec 15, 2000 at 11:18:35AM -0800, Ulrich Drepper wrote:
> > Andrea Arcangeli <andrea@suse.de> writes:
> > 
> > > x()
> > > {
> > > 
> > > 	switch (1) {
> > > 	case 0:
> > > 	case 1:
> > > 	case 2:
> > > 	case 3:
> > > 	;
> > > 	}
> > > }
> > > 
> > > Why am I required to put a `;' only in the last case and not in all
> > > the previous ones? Or maybe gcc-latest is forgetting to complain about
> > > the previous ones ;)
> > 
> > Your C language knowledge seems to have holes.  It must be possible to
> > have more than one label for a statement.  Look through the kernel
> > sources, there are definitely cases where this is needed.
> 
> I don't understand what you're talking about. Who ever talked about "more than
> one label"?
> 
> The only issue here is having 1 random label at the end of a compound
> statement. Nothing else.

The label must be on an expression. Until the ";" is present to indicate
a null expression it is syntacticly incorrect to have

switch (x) {
1:
2: something;
3:
}

The "3:" needs an expression to satisfy the syntax of "switch".

> And yes I can see that the whole point of the change is that they want
> to also forbids this:
> 
> x()
> {
> 	goto out;
> out:
> }
> 
> and I dislike not being allowed to do the above as well infact ;).

I think this has the same requirement. A null expression, specified with
the ";" is a small price to pay for simplifying the error detection.

-------------------------------------------------------------------------
Jesse I Pollard, II
Email: pollard@navo.hpc.mil

Any opinions expressed are solely my own.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/

             reply	other threads:[~2000-12-15 22:27 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2000-12-15 21:56 Jesse Pollard [this message]
2000-12-15 22:22 ` Andrea Arcangeli
  -- strict thread matches above, loose matches on Subject: below --
2000-12-15 16:29 Mike Black
2000-12-15 16:56 ` Andrea Arcangeli
2000-12-15 17:07   ` Richard B. Johnson
2000-12-15 17:43     ` Andrea Arcangeli
2000-12-15 17:59   ` Franz Sirl
2000-12-15 18:34     ` Richard B. Johnson
2000-12-15 18:50       ` Alan Cox
2000-12-15 19:06       ` Horst von Brand
2000-12-15 20:15         ` Michael Meissner
2000-12-15 18:54     ` Andrea Arcangeli
2000-12-15 19:18       ` Ulrich Drepper
2000-12-15 19:57         ` Andrea Arcangeli
2000-12-15 20:09           ` Rik van Riel
2000-12-15 20:22             ` Andrea Arcangeli
2000-12-16  7:53           ` Anuradha Ratnaweera
2000-12-16 13:52             ` Andrea Arcangeli
2000-12-19 16:53               ` Michael Meissner
2000-12-15 19:55       ` Rik van Riel
2000-12-15 20:14         ` Andrea Arcangeli
2000-12-21 18:18           ` Thomas Dodd
2001-01-02 16:55             ` Andrea Arcangeli
2000-12-15 20:31       ` Michael Meissner
2000-12-17  5:27         ` Albert D. Cahalan

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=200012152156.PAA137696@tomcat.admin.navo.hpc.mil \
    --to=pollard@tomcat.admin.navo.hpc.mil \
    --cc=andrea@suse.de \
    --cc=drepper@cygnus.com \
    --cc=linux-kernel@vger.kernel.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

Powered by JetHome