[cfe-commits] [Request for approval] Wrong extern inside no braces linkage specification declaration

Douglas Gregor dgregor at apple.com
Wed Apr 28 07:32:21 PDT 2010


On Apr 28, 2010, at 2:58 AM, Abramo Bagnara wrote:

> Currently clang marks these two declarations
> 
> extern "C" int a;
> extern "C" void foo();
> 
> as extern in StorageClassAsWritten, but the inner declarations does not
> have any storage class specified (and C++ standard does not allow any).
> 
> The attached patch wrt r102259 fixes the problem (and do not introduce
> any regressions).
> 
> This patch has already been sent to cfe-dev, currently without pending
> objections but without explicit approval.
> 
> http://lists.cs.uiuc.edu/pipermail/cfe-dev/2010-April/008773.html
> 
> Having just obtained write access, is it ok if I commit it in?

Looks good; please go ahead and commit!

	- Doug



More information about the cfe-commits mailing list