<div dir="ltr">On Sun, Sep 1, 2013 at 5:34 PM, Kim Gräsman <span dir="ltr"><<a href="mailto:kim.grasman@gmail.com" target="_blank">kim.grasman@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
  I built and tested this locally, and it works really well!<br>
<br>
  The Visual Studio SDK was necessary for building, thanks for the hint. Maybe add a README with info on prerequisites? The VS SDK for 2012 is available here:<br>
  <a href="http://www.microsoft.com/en-us/download/details.aspx?id=30668" target="_blank">http://www.microsoft.com/en-us/download/details.aspx?id=30668</a><br>
<br>
  Also, the SVN patch didn't carry the binary files Images.png and Package.ico, so I had to create dummy images, but that should work better from a proper checkout.<br>
<br>
  LGTM.<br>
<br>
<br>
================<br>
Comment at: tools/clang-format-vs/ClangFormat/PkgCmdID.cs:2<br>
@@ +1,3 @@<br>
+// PkgCmdID.cs<br>
+// MUST match PkgCmdID.h<br>
+<br>
----------------<br>
I find these comments confusing, I guess they came from a code generator?<br>
<br>
================<br>
Comment at: tools/clang-format-vs/ClangFormat/Guids.cs:2<br>
@@ +1,3 @@<br>
+// Guids.cs<br>
+// MUST match guids.h<br>
+using System;<br>
----------------<br>
I find these comments confusing, I guess they came from a code generator?<br></blockquote><div><br></div><div>Yep, those comments are from the generators - do you think I should delete all those generator-generated comments?</div>
<div><br></div></div></div></div>