<div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="im">Constifying Init requires that I get rid of non-const methods.  These</div>
are methods like BitsInit::setBit() that modify already-existing<br>
objects.  Once those are gone, various pieces of code that create Inits<br>
by cloning an existing Init and modifying it need to be updated to avoid<br>
the modification.</blockquote><div><br></div><div>This could still be split up further, simply by reversing the changes:<br><br>Update existing code that mutates objects so it they don't mutate.<br>Remove mutation functions.<br>
Add const.<br><br>(these last two can be done in parallel (but still as separate changes/CRs, so they're small & easy to consume), since they don't overlap, really) <br><br>This kind of approach also makes diff history easier to read - each change does exactly one thing & is easy to reason about & work with, should a regression occur in that particular revision.</div>
<div><br></div><div>- David</div></div>