Re: [clang-tools-extra] r280840 - Add a clang-tidy visual studio extension.

2018-10-10 Thread Aaron Ballman via cfe-commits
On Wed, Oct 10, 2018 at 5:11 PM Zachary Turner wrote: > > Yea, that’s actually another reason I suggest trying the Clang Power Tools > extension. It seems to have “won” in this area, and few people ever used the > LLVM one to begin with. Given that this isn't being updated and it may not work,

Re: [clang-tools-extra] r280840 - Add a clang-tidy visual studio extension.

2018-10-10 Thread Zachary Turner via cfe-commits
Yea, that’s actually another reason I suggest trying the Clang Power Tools extension. It seems to have “won” in this area, and few people ever used the LLVM one to begin with. On Wed, Oct 10, 2018 at 2:09 PM Aaron Ballman wrote: > On Wed, Oct 10, 2018 at 4:54 PM Zachary Turner wrote: > > > > Hon

Re: [clang-tools-extra] r280840 - Add a clang-tidy visual studio extension.

2018-10-10 Thread Aaron Ballman via cfe-commits
On Wed, Oct 10, 2018 at 4:54 PM Zachary Turner wrote: > > Honestly I hadn’t thought about this ever since the patch. Out of curiosity, > have you tried the clang power tools extension? I think it’s more actively > maintained and at this point probably even is good enough that this one could > j

Re: [clang-tools-extra] r280840 - Add a clang-tidy visual studio extension.

2018-10-10 Thread Zachary Turner via cfe-commits
Honestly I hadn’t thought about this ever since the patch. Out of curiosity, have you tried the clang power tools extension? I think it’s more actively maintained and at this point probably even is good enough that this one could just go away On Wed, Oct 10, 2018 at 1:52 PM Aaron Ballman wrote: >

Re: [clang-tools-extra] r280840 - Add a clang-tidy visual studio extension.

2018-10-10 Thread Aaron Ballman via cfe-commits
On Wed, Sep 7, 2016 at 2:37 PM Zachary Turner via cfe-commits wrote: > > Author: zturner > Date: Wed Sep 7 13:28:55 2016 > New Revision: 280840 > > URL: http://llvm.org/viewvc/llvm-project?rev=280840&view=rev > Log: > Add a clang-tidy visual studio extension. > > For now this only adds the UI nec