<div style="font-family: arial, helvetica, sans-serif"><font size="2"><div class="gmail_quote">On Wed, Jun 13, 2012 at 4:57 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Sending this for review because it has a few different changes in<br>
areas I'm less familiar with (driver) though they're mostly<br>
mechanical. Each one's probably OK for post-commit review, but I<br>
didn't want to spam the commit queue & then find review feedback for<br>
the first one that would interfere with the later patches & all get a<br>
bit messy.<br></blockquote><div><br></div><div>Can you break these up into separate patches to make review easier?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
The changes included are:<br>
<br>
Rename -rewrite-includes to -frewrite-includes<br></blockquote><div><br></div><div>Just commit this part, it's fine to post-commit review.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Support -frewrite-includes as an option while preprocessing, rather<br>
than as a flag of its own<br></blockquote><div><br></div><div>Go ahead and commit this one too, although a separate commit please.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Fix crash in -frewrite-includes when a #include specified a file that<br>
could not be found.<br></blockquote><div><br></div><div>And go ahead and commit this one, but again as a separate commit. =]</div><div><br></div><div><br></div><div>Then lets review the two left as two separate patch files?</div>
</div></font></div>