<div dir="ltr">Perhaps, but "else if (getThing().getOtherThing() == const)" ... repeated N times sounds like 'switch'.<div><div>Suggesting to use a syntax about which the compiler won't warn is like the tail wagging the dog.</div><div><br></div><div>At any rate, this was patterned after the constructor for NaCl_TC (also in ToolChains) which has the same 3-clause switch plus a default: break, and the symmetry lends to readability.</div><div><br><div class="gmail_extra"><div class="gmail_quote">On Tue, Jun 23, 2015 at 8:50 AM, Nico Weber <span dir="ltr"><<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p dir="ltr">Yes, which is probably why this was an if, not a switch (?)</p></blockquote></div></div></div></div></div>