<div dir="ltr">Did this ever land?</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sun, Mar 23, 2014 at 11:57 AM, Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk" target="_blank">richard@metafoo.co.uk</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">LGTM</div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><br><div class="gmail_quote">
On Sat, Mar 22, 2014 at 10:15 AM, Nico Rieck <span dir="ltr"><<a href="mailto:nico.rieck@gmail.com" target="_blank">nico.rieck@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>On <a href="tel:21.03.2014%2001" value="+12103201401" target="_blank">21.03.2014 01</a>:19, Richard Smith wrote:<br>

> * goes on the right.<br>
</div>> [...]<br>
<div>> 'i' and 'e' should be capitalized.<br>
<br>
</div>That's what I get for copy & paste without looking closely... ;)<br>
<div><br>
> You don't have any tests for the attribute being in the declarator chunks,<br>
> rather than in the decl specifiers. Please add such tests.<br>
<br>
</div>Done, treating all three attribute placements the same, similar to other<br>
attributes. I noticed that Clang and GCC handle such attributes<br>
differently, i.e. GCC rejects something like<br>
  int *__attribute__((visibility("hidden"))) v;<br>
<div><br>
> Move the getAttr into the assert, so we trivially remove the whole thing in<br>
> an NDEBUG build.<br>
<br>
</div>Done.<br>
<span><font color="#888888"><br>
-Nico<br>
<br>
</font></span></blockquote></div><br></div>
</div></div><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>