<div dir="ltr">LGTM</div><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 class="">On <a href="tel:21.03.2014%2001" value="+12103201401">21.03.2014 01</a>:19, Richard Smith wrote:<br>
> * goes on the right.<br>
</div>> [...]<br>
<div class="">> 'i' and 'e' should be capitalized.<br>
<br>
</div>That's what I get for copy & paste without looking closely... ;)<br>
<div class=""><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 class=""><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 class="HOEnZb"><font color="#888888"><br>
-Nico<br>
<br>
</font></span></blockquote></div><br></div>