<p dir="ltr"><br>
On 18 Apr 2014 00:37, "Daniel Marjamäki" <<a href="mailto:Daniel.Marjamaki@evidente.se">Daniel.Marjamaki@evidente.se</a>> wrote:<br>
><br>
><br>
> Hello!<br>
><br>
> > -  if (AvailabilityAttr *AA = dyn_cast<AvailabilityAttr>(Attr))<br>
> > +  if (const auto *AA = dyn_cast<AvailabilityAttr>(Attr))<br>
><br>
> Personally I don't think hiding "AvailabilityAttr" with "auto" makes the code much more readable here. In my humble opinion the auto should only be used if it really makes the code easier to read.</p>

<p dir="ltr">From the llvm coding standard:</p>
<p dir="ltr">" use <tt>auto</tt> with initializers like <tt>cast<Foo>(...)</tt> or other places where the type is already obvious from the context. "</p>
<p dir="ltr">> Best regards,<br>
> Daniel Marjamäki<br>
><br>
> ..................................................................................................................<br>
> Daniel Marjamäki Senior Engineer<br>
> Evidente ES East AB  Warfvinges väg 34  SE-112 51 Stockholm  Sweden<br>
><br>
> Mobile:                 +46 (0)709 12 42 62<br>
> E-mail:                 <a href="mailto:Daniel.Marjamaki@evidente.se">Daniel.Marjamaki@evidente.se</a><br>
><br>
> <a href="http://www.evidente.se">www.evidente.se</a><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">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</p>