[PATCH] Add clang-format VS extension

Daniel Jasper djasper at google.com
Thu Aug 29 03:49:16 PDT 2013



================
Comment at: tools/clang-format-vs/ClangFormat/ClangFormatPackage.cs:73
@@ +72,3 @@
+            var end = view.Selection.End.Position.GetContainingLine().End.Position;
+            //var offset = Math.Min(view.Selection.Start.Position.Position, text.Length-1);
+            var length = end - start;
----------------
Intentional?

================
Comment at: tools/clang-format-vs/ClangFormat/ClangFormatPackage.cs:75
@@ +74,3 @@
+            var length = end - start;
+            if (start >= text.Length && text.Length > 0)
+                start = text.Length - 1;
----------------
This might need a comment.

================
Comment at: tools/clang-format-vs/ClangFormat/PkgCmdID.cs:10
@@ +9,3 @@
+        public const uint cmdidClangFormat =        0x100;
+
+
----------------
Is this generated? Consider removing the empty lines ..


http://llvm-reviews.chandlerc.com/D1543



More information about the cfe-commits mailing list