[PATCH] Improved llvm-namespace-comment check.
Alexander Kornienko
alexfh at google.com
Mon May 19 09:54:49 PDT 2014
On Mon, May 19, 2014 at 6:53 PM, Manuel Klimek <klimek at google.com> wrote:
> On Mon, May 19, 2014 at 6:29 PM, Manuel Klimek <klimek at google.com> wrote:
>
>> lg
>>
>> ================
>> Comment at: clang-tidy/llvm/NamespaceCommentCheck.cpp:66
>> @@ +65,3 @@
>> + Token Tok;
>> + while (Lexer::getRawToken(Loc, Tok, Sources,
>> Result.Context->getLangOpts())) {
>> + Loc = Loc.getLocWithOffset(1);
>> ----------------
>> Can you add a comment explaining why it's ok to retry on failure with an
>> incremented location?
>>
>> ================
>> Comment at: clang-tidy/llvm/NamespaceCommentCheck.cpp:73
>> @@ +72,3 @@
>> + bool NextTokenIsOnSameLine = Sources.getSpellingLineNumber(Loc) ==
>> EndLine;
>> + bool NeedLineBreak = NextTokenIsOnSameLine && Tok.isNot(tok::eof);
>> +
>> ----------------
>> Can you explain why we need this? (I'd have expected the fix-it to not
>> change the number of newlines, thus just inserting the comment between the
>> } and the newline).
>>
>
> Am I missing where you added a comment for this?
>
Should be here:
http://reviews.llvm.org/D3825?vs=9577&id=9579&whitespace=ignore-all#toc
>
>>
>> http://reviews.llvm.org/D3825
>>
>>
>>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140519/f334414f/attachment.html>
More information about the cfe-commits
mailing list