[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