================ 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 _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
