[cfe-dev] Fix for clang-tidy to allow using -list-checks without positional parameters

[Alexander Kornienko via cfe-dev]

P.S. In any case, the right list for patches is cfe-commits rather than
cfe-dev, and it usually makes sense to add at least one specific reviewer
(e.g. someone who recently was making changes to the files the patch
touches).

Thanks!

On Fri, Aug 14, 2015 at 7:24 PM, Alexander Kornienko <alexfh at google.com>
wrote:

> Hi,
>
> Thank you for the patches! I find it more convenient to review patches via
> Phabricator <http://reviews.llvm.org>. If you don't mind, could you
> upload them as described here <http://llvm.org/docs/Phabricator.html>, put
> me to reviewers: and cfe-commits to subscribers? I'll take a look at the
> patches early next week.
>
> Thanks!
>
> -- Alex
>
> On Sun, Aug 9, 2015 at 1:38 AM, don hinton <hintonda at gmail.com> wrote:
>
>> Hi:
>>
>> This is a trivial fix, but I just wanted to start working with clang
>> tooling and figured this was a good way to get my feet wet.
>>
>> I've attached two patch files, one for
>> clang::tooling::CommonOptionsParser which adds a new ctor taking an
>> cl::NumOccurrences parameter for the SourcePaths option.   It defaults to
>> current behavior, so users don't need to make any changes.
>>
>> The second patch is for clangTidyMain.  It just adds a dummy file if
>> pathlist is empty, and maintains all current logic.
>>
>> thanks...
>> don
>>
>>
>>
>> _______________________________________________
>> cfe-dev mailing list
>> cfe-dev at lists.llvm.org
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20150814/b797ada3/attachment.html>