<div dir="ltr">Hi,<div><br></div><div>sending the patch here is correct or you can also try out Phabricator at <a href="http://llvm-reviews.chandlerc.com/">http://llvm-reviews.chandlerc.com/</a> which we use frequently at least for clang-format related changes.</div>
<div><br></div><div>I have tried to double-check with <a href="https://developer.mozilla.org/en-US/docs/Developer_Guide/Coding_Style">https://developer.mozilla.org/en-US/docs/Developer_Guide/Coding_Style</a> and I don't think all of you changes are correct:</div>
<div><br></div><div><div>> +  MozillaStyle.BreakConstructorInitializersBeforeComma = true;</div><div>> +  MozillaStyle.ConstructorInitializerIndentWidth = 2;</div><div><br></div><div>Yep, this seems right (<a href="https://developer.mozilla.org/en-US/docs/Developer_Guide/Coding_Style#Classes">https://developer.mozilla.org/en-US/docs/Developer_Guide/Coding_Style#Classes</a>).</div>
<div><br></div><div>> -  MozillaStyle.IndentCaseLabels = true;<br></div><div><br></div><div>I think this is incorrect. In <a href="https://developer.mozilla.org/en-US/docs/Developer_Guide/Coding_Style#Control_Structures">https://developer.mozilla.org/en-US/docs/Developer_Guide/Coding_Style#Control_Structures</a>, "case 1", "case 2" and "default" are indented.</div>
<div><br></div><div>> +  MozillaStyle.ContinuationIndentWidth = 2;</div><div><br></div><div>This is not obvious from the style guide. A continuation explicitly is not a logical level so the corresponding paragraph doesn't apply. Specifically, many source files seems to indent continuations by aligning with the open parenthesis, which is also not an indentation of 2. Am I missing a section from the style guide? If not, could we put one in? I am happy to trust you here as you have probably seen many source files.</div>
<div><br></div><div>> +  MozillaStyle.BreakBeforeBraces = FormatStyle::BS_Linux;</div><div><br></div><div>Yep. Looks right (gathered from several sections).</div></div><div><br></div><div>Cheers,</div><div>Daniel</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jan 20, 2014 at 2:00 AM, Anthony Jones <span dir="ltr"><<a href="mailto:ajones@mozilla.com" target="_blank">ajones@mozilla.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
This is my first llvm or clang patch so I apologise if I've missed some<br>
step in the process. Currently clang-format's idea of Mozilla style is<br>
substantially incorrect. This patch updates the Mozilla default to make<br>
it more closely match the style used by Mozilla. There is further work<br>
to do here.<br>
<span class="HOEnZb"><font color="#888888"><br>
Anthony<br>
</font></span><br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>