<div dir="ltr">On Wed, Nov 27, 2013 at 6:55 AM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This patch demonstrates the direction I was going to go with custom<br>
subject handling of attributes. It only handles the IBAction<br>
attribute, as that seemed like a simple place to start.<br>
<br>
When attempting to deduce the diagnostic argument, I am looking<br>
through to the SubsetSubject's Base member. However, I suspect most<br>
attributes will use the custom diagnostic logic for this.<br>
<br>
I am only generating the custom checking code once, and reusing it<br>
as-needed. Also, I am not attempting to support SubsetSubjects within<br>
SubsetSubjects at this point.</blockquote><div><br></div><div>Tiny things:</div><div><br></div><div>Maybe call the checking function is$SubjectName, not just $SubjectName?</div><div>Maybe issue an error of the SubjectList contains a SubsetSubject but no custom diagnostic, to guard against people doing that by accident? Attributes can explicitly specify the non-subset diagnostic if that's what they wanted.</div>
<div><br></div><div>Otherwise, LGTM</div></div></div></div>