<div>You mean you build a modified version of clang-format where Style is initialized to getGoogleStyle()?</div><div><br></div><div>I had wondered whether adding a -defaultStyle argument might be useful, specifically in the case where you want to pass in yaml that simply tweaks the default style, but I figured it's not much harder to pass in "BasedOnStyle" in the yaml.</div><div><br><div class="gmail_quote"><div>On Fri, Jan 20, 2017 at 8:17 AM Eric Liu via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">ioeric added a comment.<br class="gmail_msg"><br><br class="gmail_msg"><br>In <a href="https://reviews.llvm.org/D28943#651488" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D28943#651488</a>, @amaiorano wrote:<br class="gmail_msg"><br><br class="gmail_msg"><br>> In <a href="https://reviews.llvm.org/D28943#651470" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D28943#651470</a>, @ioeric wrote:<br class="gmail_msg"><br>><br class="gmail_msg"><br>> > @amaiorano: The test itself is correct. It's just that this test failed in our internal test. We could've fixed it internally, but the fix would be ugly. Since the intended behavior is already covered in the case above it, and it's really just checking the default fallback style is LLVM, which is not related to the original change, I think it makes sense to get rid of the case. Hope you don't mind :)<br class="gmail_msg"><br>><br class="gmail_msg"><br>><br class="gmail_msg"><br>> Of course I don't mind :) Why did it fail your internal tests, btw? Just curious. Was it something I could've detected myself?<br class="gmail_msg"><br><br class="gmail_msg"><br><br class="gmail_msg"><br>Probably not... it's just that our default fallback style is "Google" instead of "LLVM".<br class="gmail_msg"><br><br class="gmail_msg"><br><br class="gmail_msg"><br>Repository:<br class="gmail_msg"><br>  rL LLVM<br class="gmail_msg"><br><br class="gmail_msg"><br><a href="https://reviews.llvm.org/D28943" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D28943</a><br class="gmail_msg"><br><br class="gmail_msg"><br><br class="gmail_msg"><br><br class="gmail_msg"><br></blockquote></div></div>