<div dir="ltr">On Wed, Apr 17, 2013 at 9:00 AM, Alexander Kornienko <span dir="ltr"><<a href="mailto:alexfh@google.com" target="_blank">alexfh@google.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><p dir="ltr"><br>
On Apr 17, 2013 9:35 AM, "Manuel Klimek" <<a href="mailto:klimek@google.com" target="_blank">klimek@google.com</a>> wrote:<br>
><br>
><br>
><br>
> ================<br>
> Comment at: lib/Format/UnwrappedLineParser.h:84<br>
> @@ +83,3 @@<br>
> +  /// \brief Returns actual token start location without leading escaped<br>
> +  /// newlines.<br>
> +  SourceLocation getRealLocation() const {<br>
> ----------------<br>
> It's not only escaped newlines, but all whitespace, right?<br>
><br>
><br>
> <a href="http://llvm-reviews.chandlerc.com/D682" target="_blank">http://llvm-reviews.chandlerc.com/D682</a></p>
</div></div><p dir="ltr">Whitespace is not part of a token anyway. <br></p></blockquote><div><br></div><div style>Yes, whitespace is not part of the Token, but it's part of the FormatToken. Since we refer to both as "token" in speech, this comment (inside the context of FormatToken) is highly misleading. FormatToken always contained all preceeding whitespace. I think the comment needs more context, and I'd rename it to something different. The problem is again that getTokenLocation() is ambiguous, as it's unclear whether it's the format token or the lexer's token.</div>
</div><br></div></div>