<div dir="ltr">That's what I wanted to ask you. We didn't have this gigantic switch and lots of break's there but a series of small ifs. You did converted them as a part of unrelated change (which is the large InputGraph patch) without any explanation. Yes, the original code looks better to me, and I guess that's at least part of the reason why the original code was written as it was. I'm basically trying to restore the original code.<div class="gmail_extra"><br><div class="gmail_quote">On Fri, Feb 6, 2015 at 7:40 PM, Shankar Kalpathi Easwaran <span dir="ltr"><<a href="mailto:shankarke@gmail.com" target="_blank">shankarke@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I really dont understand, Its a matter of style, every developer may choose to follow different styles. Nothing is written in the coding convention document about what style to choose over another.<br>
<div><div><br>
<br>
<a href="http://reviews.llvm.org/D7478" target="_blank">http://reviews.llvm.org/D7478</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</div></div></blockquote></div><br></div></div>