Hi Jordy,<div><br><div class="gmail_quote">On Fri, Apr 27, 2012 at 4:22 AM, Jordy Rose <span dir="ltr"><<a href="mailto:jediknil@belkadan.com">jediknil@belkadan.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Oh, and...<br>
<br>
> +#include <set><br>
<br>
This seems unused now.<br>
<br>
> +    static const AttributedStmt *AsFallThroughAttr(const Stmt *S) {<br>
> +      if (const AttributedStmt *AS = dyn_cast_or_null<AttributedStmt>(S)) {<br>
> +        for (AttrVec::const_iterator It = AS->getAttrs().begin(),<br>
> +                                     End = AS->getAttrs().end();<br>
> +                                     It != End; ++It) {<br>
> +          if (isa<FallThroughAttr>(*It))<br>
> +            return AS;<br>
> +        }<br>
> +      }<br>
> +      return 0;<br>
> +    }<br>
<br>
The inner for-loop can be replaced with hasSpecificAttr<FallThroughAttr>(AS->getAttrs());.</blockquote></div><div><br></div><div>Thanks for noting! Fixed. Will post in the next patch.<br></div><br>-- 
</div><div>Best regards,</div><div>Alexander Kornienko</div>