<p dir="ltr">I see.. I think we'll want to keep the current behavior there.. </p>
<div class="gmail_quote">On Aug 27, 2015 3:38 PM, "Beren Minor" <<a href="mailto:beren.minor%2Bgithub@gmail.com">beren.minor+github@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Alright, my bad. It does indeed.<br><br></div>I was trying to add a "Reformat all on save" feature in the plugin, and after struggling with VSSDK I thought this would be an easy first step.<br><div class="gmail_extra"><br clear="all"><div><div>--<br>Beren Minor<br></div></div>
<br><div class="gmail_quote">On Thu, Aug 27, 2015 at 3:36 PM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On Thu, Aug 27, 2015 at 9:34 AM, Daniel Jasper via cfe-commits<br>
<<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>> wrote:<br>
> If nothing is selected, clang-format should format the current line.. At<br>
> least that's the intended behavior. Doesn't it do that?<br>
<br>
</span>It currently reformats the current line (possibly extended if the<br>
expression spans multiple lines) for me.<br>
<br>
~Aaron<br>
<div><div><br>
><br>
> On Aug 27, 2015 3:21 PM, "Beren Minor" <<a href="mailto:beren.minor%2Bgithub@gmail.com" target="_blank">beren.minor+github@gmail.com</a>> wrote:<br>
>><br>
>> berenm created this revision.<br>
>> berenm added a reviewer: djasper.<br>
>> berenm added a subscriber: cfe-commits.<br>
>><br>
>> By default, clang-format VS plugin only reformats the selected code.<br>
>><br>
>> To reformat the whole document, the user has to select everything before<br>
>> calling the reformat shortcut.<br>
>><br>
>><br>
>> <a href="http://reviews.llvm.org/D12405" rel="noreferrer" target="_blank">http://reviews.llvm.org/D12405</a><br>
>><br>
>> Files:<br>
>>   tools/clang-format-vs/ClangFormat/ClangFormatPackage.cs<br>
>><br>
>> Index: tools/clang-format-vs/ClangFormat/ClangFormatPackage.cs<br>
>> ===================================================================<br>
>> --- tools/clang-format-vs/ClangFormat/ClangFormatPackage.cs<br>
>> +++ tools/clang-format-vs/ClangFormat/ClangFormatPackage.cs<br>
>> @@ -84,8 +84,13 @@<br>
>>                  // We're not in a text view.<br>
>>                  return;<br>
>>              string text = view.TextBuffer.CurrentSnapshot.GetText();<br>
>> -            int start =<br>
>> view.Selection.Start.Position.GetContainingLine().Start.Position;<br>
>> -            int end =<br>
>> view.Selection.End.Position.GetContainingLine().End.Position;<br>
>> +            int start = 0;<br>
>> +            int end = text.Length;<br>
>> +            if (!view.Selection.IsEmpty)<br>
>> +            {<br>
>> +                start =<br>
>> view.Selection.Start.Position.GetContainingLine().Start.Position;<br>
>> +                end =<br>
>> view.Selection.End.Position.GetContainingLine().End.Position;<br>
>> +            }<br>
>>              int length = end - start;<br>
>>              // clang-format doesn't support formatting a range that<br>
>> starts at the end<br>
>>              // of the file.<br>
>><br>
>><br>
><br>
</div></div>> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a><br>
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
><br>
</blockquote></div><br></div></div>
</blockquote></div>