<div dir="ltr"><div><div>The tests that for making sure warnings are issued when attributes are attached to the wrong kind of thing aren't in warn-consumed-analysis.cpp.  Like I said, they are in warn-consumed-parsing.cpp.<br>
<br></div>I'm splitting the patch now.<br><br></div>- Chris<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Aug 21, 2013 at 10:25 AM, 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"><div class="im">On Wed, Aug 21, 2013 at 12:39 PM, Christian Wailes<br>
<<a href="mailto:chriswailes@google.com">chriswailes@google.com</a>> wrote:<br>
> With regards to the tests, those already exist.  They are in the<br>
> warn-consumed-parsing.cpp file.  As for the switch statements, I'll move<br>
> those to fully-covered switches.<br>
<br>
</div>I do not see any such tests in SemaCXX\warn-consumed-analysis.cpp.<br>
The only tests I see there are testing other semantics; eg) invocation<br>
<div class="im">of method 'operator*' on object 'var' while it is in the 'consumed'<br>
state<br>
<br>
</div><div class="im">> I was not claiming that keeping patches limited is a bad idea, or was of<br>
> limited benefit in general.  Yes, I should have submitted this as multiple<br>
> patches, but to go back now and dig the different parts out of the commit<br>
> out would be time consuming, for very little benefit in this case.  All of<br>
> the things touched are inside my experimental code, or are the removal of<br>
> simple asserts.  I'd really like to focus on producing actual work, and keep<br>
> this as a lesson for next time.<br>
<br>
</div>I understand the desire to move forward, but splitting patches into<br>
individual units of work is a clang patch submission criteria.  I<br>
believe it does have utility in this case since at least one of those<br>
patches would already be accepted (the assert removals) while the<br>
other two patches could have better-focused discussion involving the<br>
proper reviewers.  Please split this into multiple patches and we can<br>
continue to review the fixes you've made.<br>
<br>
Thanks!<br>
<span class="HOEnZb"><font color="#888888"><br>
~Aaron<br>
</font></span></blockquote></div><br></div>