byteToColumn and columnToByte bodies probably shouldn't be buried inside SourceColumnMap. The indentation is confusing because of it.<br><br>Believe the general still is to keep the operator at the end of the previous line when splitting lines<br>


<br>

+  assert(CaretStart!=(unsigned)-1 && CaretEnd!=(unsigned)-1<br>

+         && SourceStart!=(unsigned)-1 && SourceEnd!=(unsigned)-1);<br>

<br>

+    std::pair<SmallString<16>,bool> res<br>

+        = printableTextForNextCharacter(line, &i, DiagOpts.TabStop);<br>



<br>

<br>


This is a nit, but the 'else' isn't needed since the 'if' ends in a return.<br>


<br>


+      return std::make_pair(expandedCP, false);<br>


+    } else {<br>


+      // If next character is valid UTF-8, and printable<br>


+      return std::make_pair(SmallString<16>(original_begin, cp_end), true);<br>


+    }<br>

<br>

~Craig<br><br><div class="gmail_quote">On Mon, Apr 16, 2012 at 8:12 PM, Seth Cantrell <span dir="ltr"><<a href="mailto:seth.cantrell@gmail.com">seth.cantrell@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Oh, and the Windows build finished and looks fine.<br>
<div><div class="h5"><br>
<br>
On Apr 16, 2012, at 11:10 PM, Seth Cantrell wrote:<br>
<br>
> It was a line length problem, but one of the other changes eliminated it.<br>
><br>
> On Apr 16, 2012, at 11:05 PM, Craig Topper wrote:<br>
><br>
>> This style is kind of hard to read as it makes arguments look like local variables at first glance. If it's a line length problem, putting "static void" on a separate line is probably a better choice so that the argument can still line up with the the parenthese.<br>

>><br>
>> +static void selectInterestingSourceRegion(<br>
>> +  std::string &SourceLine,<br>
>> +  std::string &CaretLine,<br>
>> +  std::string &FixItInsertionLine,<br>
>> +  unsigned Columns,<br>
>> +  const SourceColumnMap &map) {<br>
>><br>
>> --<br>
>> ~Craig<br>
</div></div>> <0003-fix-parameter-formatting.patch><br>
<br>
</blockquote></div><br><br clear="all"><br>-- <br>~Craig<br>