<div dir="ltr">LGTM. I assume you reversed the visitation order for superclasses to visit the most-derived class first?</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Nov 20, 2013 at 3:55 PM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Attr.td has the notion of a DefaultIntArgument that currently does<br>
nothing special. This patch adds table generator support such that<br>
attributes which use the DefaultIntArgument type are given a static<br>
const class member that names the default value.  Eg)<br>
<br>
let Args = [DefaultIntArgument<"Priority", 65535>];<br>
<br>
Would give the generated attribute class a public member:<br>
<br>
static const int DefaultPriority = 65535;<br>
<br>
This can then be used in place of magic numbers elsewhere (and<br>
consequently, this patch removes a few such FIXMEs).<br>
<span class="HOEnZb"><font color="#888888"><br>
~Aaron<br>
</font></span></blockquote></div><br></div>