[PATCH] run clang-tidy namespace-comment check on LLVM

Alexander Kornienko alexfh at google.com
Mon Jun 22 02:57:56 PDT 2015


And the same thing for clang is committed in r240270.

-- Alex
On Fri, Jun 19, 2015 at 6:03 PM, Alexander Kornienko <alexfh at google.com>
wrote:

> Anyway, ran clang-tidy once again and committed as r240137.
>
> Thanks for the patch!
>
>
> On Fri, Jun 19, 2015 at 2:47 PM, Alexander Kornienko <alexfh at google.com>
> wrote:
>
>> I like the idea, but there are suspiciously too few changes (I have ~20x
>> more). How did you run clang-tidy and which options did you use?
>>
>> On Fri, Jun 5, 2015 at 1:07 PM, Eugene Kosov <claprix at yandex.ru> wrote:
>>
>>> Hello.
>>>
>>> Patch is pretty simple. Please, commit if all is fine.
>>>
>>> --
>>> Eugene
>>>
>>> _______________________________________________
>>> llvm-commits mailing list
>>> llvm-commits at cs.uiuc.edu
>>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>>>
>>>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150622/594d1496/attachment.html>


More information about the llvm-commits mailing list