<div dir="ltr">Thanks Richard!<div><br></div><div>Committed with getAs<> fix as r224184. Fixed the * placement in r224185 due to lack of attention on my part....</div><div><br></div><div>Will.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On 13 December 2014 at 08:46, 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">+  if (const EnumType* ET = dyn_cast<EnumType>(T)) {<br><div><br></div><div>* on the right please, and use T->getAs<EnumType>() in case there is type sugar here (there probably can't be for a TemplateArgument that's been resolved to an integral value, but dyn_cast on a type is a red flag that we don't need here).</div><div><br></div><div>Otherwise, LGTM, thanks!</div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 11, 2014 at 8:28 PM, Will Wilson <span dir="ltr"><<a href="mailto:will@indefiant.com" target="_blank">will@indefiant.com</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">Hi Richard,
<div><br></div><div>Here's a simple patch that improves pretty printing for templates args specialized to enum constants. If an enum constant isn't available to represent the value it falls back on printing the integer value.</div><div><br></div><div>Tested against latest trunk and includes a new test case and a updated existing test case.</div><div><br></div><div>Let me know if it looks good to go...</div><div><br></div><div>Cheers,</div><div>Will.</div></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><span style="background-color:rgb(255,255,255);color:rgb(68,68,68);font-family:Arial,Helvetica,sans-serif"><b>Indefiant </b>: <a href="http://www.indefiant.com" target="_blank">http://www.indefiant.com</a></span></div><div><span style="font-family:Arial,Helvetica,sans-serif"><font color="#999999">Home of Recode : Runtime C++ Editing for VS</font></span></div></div></div>
</div>