[PATCH] D12405: [clang-format-vs] Format the whole document if nothing is selected

[Beren Minor via cfe-commits]

berenm created this revision.
berenm added a reviewer: djasper.
berenm added a subscriber: cfe-commits.

By default, clang-format VS plugin only reformats the selected code.

To reformat the whole document, the user has to select everything before 
calling the reformat shortcut.


http://reviews.llvm.org/D12405

Files:
  tools/clang-format-vs/ClangFormat/ClangFormatPackage.cs

Index: tools/clang-format-vs/ClangFormat/ClangFormatPackage.cs
===================================================================
--- tools/clang-format-vs/ClangFormat/ClangFormatPackage.cs
+++ tools/clang-format-vs/ClangFormat/ClangFormatPackage.cs
@@ -84,8 +84,13 @@
                 // We're not in a text view.
                 return;
             string text = view.TextBuffer.CurrentSnapshot.GetText();
-            int start = 
view.Selection.Start.Position.GetContainingLine().Start.Position;
-            int end = 
view.Selection.End.Position.GetContainingLine().End.Position;
+            int start = 0;
+            int end = text.Length;
+            if (!view.Selection.IsEmpty)
+            {
+                start = 
view.Selection.Start.Position.GetContainingLine().Start.Position;
+                end = 
view.Selection.End.Position.GetContainingLine().End.Position;
+            }
             int length = end - start;
             // clang-format doesn't support formatting a range that starts at 
the end
             // of the file.



Index: tools/clang-format-vs/ClangFormat/ClangFormatPackage.cs
===================================================================
--- tools/clang-format-vs/ClangFormat/ClangFormatPackage.cs
+++ tools/clang-format-vs/ClangFormat/ClangFormatPackage.cs
@@ -84,8 +84,13 @@
                 // We're not in a text view.
                 return;
             string text = view.TextBuffer.CurrentSnapshot.GetText();
-            int start = view.Selection.Start.Position.GetContainingLine().Start.Position;
-            int end = view.Selection.End.Position.GetContainingLine().End.Position;
+            int start = 0;
+            int end = text.Length;
+            if (!view.Selection.IsEmpty)
+            {
+                start = view.Selection.Start.Position.GetContainingLine().Start.Position;
+                end = view.Selection.End.Position.GetContainingLine().End.Position;
+            }
             int length = end - start;
             // clang-format doesn't support formatting a range that starts at the end
             // of the file.

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits