<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">On Mon, May 6, 2013 at 10:06 AM, Alexander Kornienko <span dir="ltr"><<a href="mailto:alexfh@google.com" target="_blank">alexfh@google.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">No, it doesn't. The only thing that bothers me a bit is that we don't have any tests for this style. There's plenty of tests for LLVM style in form of unit tests, but we still need some kind of file-based integration tests for different style guides.<div>

</div></div></blockquote><div><br></div><div>I think it might make more sense to have tests for individual style properties.  That way we wouldn't need to add ad-hoc tests for every project's style.  But I would expect that doing that should be easier once the configuration file work you mentioned in done.<br>

</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Anyway, I'm going to commit your patch with getMozillaStyle being based upon getLLVMStyle: <a href="http://llvm-reviews.chandlerc.com/D750" target="_blank">http://llvm-reviews.chandlerc.com/D750</a>. If you have any comments, you can share them on Phabricator.</div>

</div></blockquote><div><br></div><div>I think the modifications are fine.  I personally prefer styles to not be dependent on each other since otherwise a modification to getLLVMStyle may mean that the Mozilla style is now different too, but the number of options are quite small right now so I don't expect that to turn into a problem in practice.<br>

</div><div><br></div><div>Thanks!<br clear="all"></div><div><div>--<br>Ehsan<br><<a href="http://ehsanakhgari.org/">http://ehsanakhgari.org/</a>></div> </div></div></div></div>