<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Mon, Dec 5, 2016 at 3:53 PM Antonio Maiorano <<a href="mailto:amaiorano@gmail.com">amaiorano@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr" class="gmail_msg"><div dir="ltr" class="gmail_msg">Hi Manuel,<div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">Okay, I'll submit both patches via phabricator as you recommend. I'll add you, and I'll add Hans Wennborg and Zachary Turner, who were both involved with the first clang-format-vsix patch I worked on: <a href="http://lists.llvm.org/pipermail/cfe-commits/Week-of-Mon-20161128/178569.html" class="gmail_msg" target="_blank">http://lists.llvm.org/pipermail/cfe-commits/Week-of-Mon-20161128/178569.html</a></div><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">Does that make sense? I'm still learning the common workflow for submitting patches :)</div></div></div></blockquote><div><br></div><div>Yep, perfect. Thanks!!</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr" class="gmail_msg"><div dir="ltr" class="gmail_msg"><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">- Antonio</div><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg"><br class="gmail_msg"></div></div></div><div dir="ltr" class="gmail_msg"><br class="gmail_msg"><div class="gmail_quote gmail_msg"><div dir="ltr" class="gmail_msg">On Mon, 5 Dec 2016 at 06:24 Manuel Klimek <<a href="mailto:klimek@google.com" class="gmail_msg" target="_blank">klimek@google.com</a>> wrote:<br class="gmail_msg"></div><blockquote class="gmail_quote gmail_msg" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr" class="gmail_msg">Awesome, thanks for working on this. If you submit the patches via phabricator (<a href="http://llvm.org/docs/Phabricator.html" class="gmail_msg" target="_blank">http://llvm.org/docs/Phabricator.html</a>) I'll get to it faster :) Feel free to directly add me as reviewer, btw (add 'klimek').</div><br class="gmail_msg"><div class="gmail_quote gmail_msg"></div><div class="gmail_quote gmail_msg"><div dir="ltr" class="gmail_msg">On Sat, Dec 3, 2016 at 6:53 AM Antonio Maiorano via cfe-commits <<a href="mailto:cfe-commits@lists.llvm.org" class="gmail_msg" target="_blank">cfe-commits@lists.llvm.org</a>> wrote:<br class="gmail_msg"></div></div><div class="gmail_quote gmail_msg"><blockquote class="gmail_quote gmail_msg" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr" class="gmail_msg">Hello,<div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">I have two patches for clang-format-vsix to offer up for review:</div><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">0001-clang-format-vsix-versioning.patch<br class="gmail_msg"></div><div class="gmail_msg"><div class="gmail_msg">clang-format-vsix: add a date stamp to the VSIX version number to ensure upgradability</div><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">Presently, the version number of the VSIX matches the LLVM version number. However, as this number doesn't change often, it means that as we release new versions of this VSIX, it will have the same version number, which means users must first uninstall the old version before installing the new one. With this change, we generate a 4th part to the version number that is a date stamp (year, month, day); for example: 4.0.0.161203.<br class="gmail_msg"></div></div><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">0001-clang-format-vsix-show-stderrs.patch<br class="gmail_msg"></div><div class="gmail_msg"><div class="gmail_msg">clang-format-vsix: fail when clang-format outputs to stderr</div><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">When clang-format outputs to stderr but returns 0, the extension will format the code anyway. This happens, for instance, when there's a syntax error or unknown value in a .clang-format file; the result is that the extension silently formats using the fallback style without informing the user of the problem. This change treats stderr output as an error, making sure it gets displayed to the user, and not formatting the code. (I've also attached a screenshot of the message box I now get when my .clang-format has an invalid field named SomeInvalidField).</div><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg"><br class="gmail_msg"></div></div></div></blockquote></div><div class="gmail_quote gmail_msg"><blockquote class="gmail_quote gmail_msg" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
_______________________________________________<br class="gmail_msg">
cfe-commits mailing list<br class="gmail_msg">
<a href="mailto:cfe-commits@lists.llvm.org" class="gmail_msg" target="_blank">cfe-commits@lists.llvm.org</a><br class="gmail_msg">
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" class="gmail_msg" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br class="gmail_msg">
</blockquote></div>
</blockquote></div></div></blockquote></div></div>