[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