mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Mitchell Blank Jr <mitch@sfgoth.com>
To: Christoph Hellwig <hch@infradead.org>,
	Stephen Hemminger <shemminger@osdl.org>,
	David Mosberger-Tang <davidm@hpl.hp.com>,
	linux-ia64@linuxia64.org, linux-kernel@vger.kernel.org
Subject: Re: GCC nested functions?
Date: Wed, 12 May 2004 17:07:17 -0700	[thread overview]
Message-ID: <20040513000712.GA29796@gaz.sfgoth.com> (raw)
In-Reply-To: <20040512190454.A31410@infradead.org>

Christoph Hellwig wrote:
> nested function are a horrible gcc misfeature.

True.

> So far people had enough
> taste to not introduce them without explicitly forbidding it ;-)

Almost true.  Look at drivers/atm/horizon.c:make_rate(), which includes
a nested function called "set_cr()"  There may be a couple other
examples in the drivers/atm directory, not sure.

The Intel CC guys made special mention of the ATM drivers when they
first started using their compiler for the kernel (icc supports most
gcc extensions, but apparently not nested functions) and they made special
mention of not being able to compile some ATM drivers because of this.

I'm not aware of nested functions anywhere else in the kernel though.

-Mitch

  reply	other threads:[~2004-05-13  0:06 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-05-12 17:59 Stephen Hemminger
2004-05-12 18:04 ` Christoph Hellwig
2004-05-13  0:07   ` Mitchell Blank Jr [this message]
2004-05-12 18:11 ` William Lee Irwin III
2004-05-12 18:13 ` David Mosberger
2004-05-12 19:30 ` viro
2004-05-12 19:34 ` Gabriel Paubert

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=20040513000712.GA29796@gaz.sfgoth.com \
    --to=mitch@sfgoth.com \
    --cc=davidm@hpl.hp.com \
    --cc=hch@infradead.org \
    --cc=linux-ia64@linuxia64.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=shemminger@osdl.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

all inboxes | Powered by JetHome®