<div dir="ltr"><div>@@ -2898,7 +2898,8 @@ void Sema::MergeVarDecl(VarDecl *New, LookupResult &Previous,</div><div> </div><div>   // C99 6.2.2p4: Check if we have a static decl followed by a non-static.</div><div>   if (New->getStorageClass() == SC_Static &&</div>
<div>-      (Old->getStorageClass() == SC_None || Old->hasExternalStorage())) {</div><div>+      !New->isStaticDataMember() &&</div><div>+      isExternalLinkage(Old->getLinkage())) {</div><div><br></div>
<div style>This looks backwards -- is the comment here wrong? The 'isStaticDataMember' check looks like it might be unnecessary, since a redeclaration of a static data member can't be marked 'static'.</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Apr 4, 2013 at 11:17 AM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This is a  bit of patching that survived 178663. Without it we can<br>
produce better a better error message for<br>
<br>
const int a = 5;<br>
static const int a;<br>
<br>
Cheers,<br>
Rafael<br>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>