aaron.ballman (Aaron Ballman)
User

Projects

User does not belong to any projects.

User Details

User Since
Mar 14 2013, 3:16 PM (223 w, 2 d)

Recent Activity

Thu, Jun 22

aaron.ballman accepted D33333: Emit warning when throw exception in destruct or dealloc functions which has a (possible implicit) noexcept specifier.

Aside from a few small nits with the test cases, this LGTM! You should hold off on committing for a day or two in case Richard or others have comments on the patch.

Thu, Jun 22, 5:25 AM

Wed, Jun 21

aaron.ballman added inline comments to D34449: [clang-tidy] Enable constexpr definitions in headers. .
Wed, Jun 21, 7:35 AM · Restricted Project

Fri, Jun 16

aaron.ballman added inline comments to D33333: Emit warning when throw exception in destruct or dealloc functions which has a (possible implicit) noexcept specifier.
Fri, Jun 16, 1:59 PM
aaron.ballman added inline comments to D33333: Emit warning when throw exception in destruct or dealloc functions which has a (possible implicit) noexcept specifier.
Fri, Jun 16, 1:46 PM
aaron.ballman added a comment to D33333: Emit warning when throw exception in destruct or dealloc functions which has a (possible implicit) noexcept specifier.

A few more nits, but this feels like it's getting close to ready (at least, to me).

Fri, Jun 16, 1:06 PM
aaron.ballman added a comment to D32332: Add support for transparent overloadable functions in clang.

This is generally looking good to me, with a few small nits. @rsmith, do you have thought on this?

Fri, Jun 16, 6:55 AM
aaron.ballman added a comment to D33563: Track whether a unary operation can overflow.

Ping

Fri, Jun 16, 6:27 AM
aaron.ballman added inline comments to D33333: Emit warning when throw exception in destruct or dealloc functions which has a (possible implicit) noexcept specifier.
Fri, Jun 16, 6:15 AM
aaron.ballman added inline comments to D33722: [clang-tidy] Add checker for undelegated copy of base classes.
Fri, Jun 16, 5:26 AM · Restricted Project

Thu, Jun 15

aaron.ballman added inline comments to D34091: Support for querying the exception specification type through libclang.
Thu, Jun 15, 2:27 PM

Wed, Jun 14

aaron.ballman closed D34179: [Frontend] PR32318 Handle -ast-dump-all properly..

Committed in r305432. You should consider reaching out to Chris Lattner for svn access!

Wed, Jun 14, 5:01 PM
aaron.ballman accepted D34179: [Frontend] PR32318 Handle -ast-dump-all properly..

LGTM!

Wed, Jun 14, 4:39 PM
aaron.ballman added a comment to D33904: Add a __has_attribute_enhancement macro to clang.

Why not just use __has_feature(overloadable_unmarked) or similar?

My impression was that __has_feature was was for larger features than tweaks to attributes. If this would be an appropriate use of __has_feature, though, I'm happy to keep things simple.

I'll update the other review with __has_feature. If it goes in with that, I'll abandon this.

Wed, Jun 14, 4:38 PM
aaron.ballman added inline comments to D34091: Support for querying the exception specification type through libclang.
Wed, Jun 14, 4:36 PM
aaron.ballman added a comment to rL305025: [ASTMatchers] temporary disable tests with floating suffix.

It turns out that these tests are failing because:

  • 1.2f == 1.2 is false, apparently 1.2 cannot be exactly represented as single-precision float. Without modifying the grammar, one can use floatLiteral(equals(1.2000000476837158203125)) to match 1.2f.
Wed, Jun 14, 4:29 PM
aaron.ballman requested changes to D33722: [clang-tidy] Add checker for undelegated copy of base classes.
Wed, Jun 14, 4:25 PM · Restricted Project
aaron.ballman added inline comments to D33841: [clang-tidy] redundant keyword check.
Wed, Jun 14, 4:08 PM · Restricted Project

Tue, Jun 13

aaron.ballman added a reviewer for D33333: Emit warning when throw exception in destruct or dealloc functions which has a (possible implicit) noexcept specifier: rsmith.

Adding Richard to the review for some wider perspective than just mine on the overall design.

Tue, Jun 13, 2:56 PM

Thu, Jun 8

aaron.ballman accepted D33094: [ASTMatchers] Add clang-query support for equals matcher.

LGTM!

Thu, Jun 8, 2:04 PM
aaron.ballman accepted D33135: [ASTMatchers] Add support for floatLiterals.

LGTM!

Thu, Jun 8, 2:01 PM
aaron.ballman accepted D33735: [DebugInfo] Add ThisOrSelf attribute for emission of FlagObjectPointer..

Aside from one minor nit, LGTM!

Thu, Jun 8, 1:56 PM

Wed, Jun 7

aaron.ballman accepted D34002: [clang-tidy] When" -fno-exceptions is used", this warning is better to be suppressed..

Aside from one minor nit, LGTM

Wed, Jun 7, 11:47 AM · Restricted Project
aaron.ballman added inline comments to D33735: [DebugInfo] Add ThisOrSelf attribute for emission of FlagObjectPointer..
Wed, Jun 7, 7:55 AM
aaron.ballman added inline comments to D33735: [DebugInfo] Add ThisOrSelf attribute for emission of FlagObjectPointer..
Wed, Jun 7, 7:43 AM
aaron.ballman accepted D33917: [clang-tidy] When" -fno-exceptions is used", this warning is better to be suppressed..

Aside from a minor suggestion, LGTM!

Wed, Jun 7, 6:38 AM · Restricted Project

Mon, Jun 5

aaron.ballman added inline comments to D33829: [clang-tidy] avoid reserved names check.
Mon, Jun 5, 1:44 PM · Restricted Project
aaron.ballman added a comment to D33825: [clang-tidy] signal handler must be plain old function check.

This check is an interesting one. The rules around what is signal safe are changing for C++17 to be a bit more lenient than what the rules are for C++14. CERT's rule is written against C++14, and so the current behavior matches the rule wording. However, the *intent* of the rule is to ensure that only signal-safe functionality is used from a signal handler, and so from that perspective, I can imagine a user compiling for C++17 to want the relaxed rules to still comply with CERT's wording. What do you think?

Mon, Jun 5, 6:11 AM · Restricted Project
aaron.ballman added a comment to D33826: [clang-tidy] avoid pointer cast to more strict alignment check.

Thank you for working on this. How does this check compare with the -Wcast-align diagnostic in the Clang frontend? I believe that warning already covers these cases, so I'm wondering what extra value is added with this check?

Mon, Jun 5, 5:36 AM · Restricted Project

Fri, Jun 2

aaron.ballman added a comment to D33788: Return a canonical path from getClangResourcePath().

I'm unaware of any other API that canonicalizes the path, which is what users of this API are going to expect.

You can also call sys::path::remove_dots(Path, /*remove_dot_dot=*/true);

Fri, Jun 2, 11:39 AM
aaron.ballman added a comment to D33788: Return a canonical path from getClangResourcePath().

Getting the real path is notoriously slow (at least it's horrible in OSX, not sure about linux). Since this is about dropping the '/../' part, could we do some simple canonicalization of removing the dots ? Not sure if there is an existing function that does that.

Fri, Jun 2, 9:38 AM
aaron.ballman added a comment to D33537: [clang-tidy] Exception Escape Checker.

I think we should try to get as much of this functionality in D33333 as possible; there is a considerable amount of overlap between that functionality and this functionality. This check can then cover only the parts that are not reasonably handled by the frontend check instead of duplicating diagnostics the user already receives.

I suppose that the frontend check will not travarse the call-graph just check direct throws. Should we only check indirect throws then?

Fri, Jun 2, 5:34 AM · Restricted Project
aaron.ballman added a comment to D33788: Return a canonical path from getClangResourcePath().

Hi Aaron,

Nice catch! Any chance you can add a testcase to this?

Fri, Jun 2, 5:33 AM

Thu, Jun 1

aaron.ballman resigned from D12407: [clang-format-vs] Add an option to reformat source code when file is saved to disk.
Thu, Jun 1, 11:01 AM
aaron.ballman abandoned D12375: [PATCH] Relax parse ordering rules for attributes.
Thu, Jun 1, 10:52 AM
aaron.ballman abandoned D15456: [PATCH] New diagnostic for non-idiomatic copy or move operations (v2).
Thu, Jun 1, 10:50 AM
aaron.ballman added a comment to D33735: [DebugInfo] Add ThisOrSelf attribute for emission of FlagObjectPointer..

Can you help me to understand what problem is being solved with this new attribute? Under what circumstances would the first argument be an ImplicitParamDecl but not an implicit this or self?

For captured regions an outlined function is created, where all parameters are ImplicitParamDecls. And the very first parameter is wrongly treated as 'this' argument of the member function.

Thu, Jun 1, 10:33 AM
aaron.ballman created D33788: Return a canonical path from getClangResourcePath().
Thu, Jun 1, 10:30 AM
aaron.ballman added a comment to D33735: [DebugInfo] Add ThisOrSelf attribute for emission of FlagObjectPointer..

Can you help me to understand what problem is being solved with this new attribute? Under what circumstances would the first argument be an ImplicitParamDecl but not an implicit this or self?

Thu, Jun 1, 10:11 AM
aaron.ballman added a comment to D33537: [clang-tidy] Exception Escape Checker.

How is that compared to https://reviews.llvm.org/D19201 and the clang patch mentioned in this patch?

Actually, this check does much more. It does not only check for noexcept (and throw()) functions, but also for destructors, move constructors, move assignments, the main() function, swap() functions and also functions given as option. A more important difference is that we traverse the whole call-chain and check all the throw statements and try-catch blocks so indirectly throwing functions are also reported and no flase positives are caused by throw and catch in the same try block.

Thu, Jun 1, 10:01 AM · Restricted Project
aaron.ballman added a comment to D33333: Emit warning when throw exception in destruct or dealloc functions which has a (possible implicit) noexcept specifier.
In D33333#768332, @jyu2 wrote:

Okay this CFG version of this change. In this change I am basic using same algorithm with -Winfinite-recursion.

In addition to my original implementation, I add handler type checking which basic using https://reviews.llvm.org/D19201 method.

Thu, Jun 1, 9:55 AM

Tue, May 30

aaron.ballman added inline comments to D33563: Track whether a unary operation can overflow.
Tue, May 30, 9:57 AM
aaron.ballman updated the diff for D33563: Track whether a unary operation can overflow.

Addressing review comments.

Tue, May 30, 9:57 AM
aaron.ballman accepted D33398: Mangle __unaligned in Itanium ABI.

Can you run clang-format over both the test files? Aside from that, looks good to me, but you should wait for @rsmith or @majnemer to sign off before committing.

Tue, May 30, 6:29 AM
aaron.ballman added a comment to D33651: Add note for location of other convenience functions..

Can you add some unit tests?

Tue, May 30, 6:23 AM
aaron.ballman added a comment to D33531: Clang-tidy readability: avoid const value return.

Would it make sense to silence this diagnostic in the presence of also checking for cert-dcl21-cpp for such operators?

Currently there's no mechanism in clang-tidy to express dependencies or compatibility issues between checks. Moreover, we already have checks that are not meant to be run together, for example, readability-braces-around-statements and its google- incarnation (and other alias checks with different settings). That said, we could whitelist postfix increment and decrement operators in this check. Camillo, WDYT?

I can imagine a generic whitelist mechanism might be useful for this check. It could even be empty by default, but the documentation could call out cert-dcl21-cpp specifically and show an example of how you can run both checks.

A generic whitelist of method/function names would make sense, if we had more use cases for it. It might also be quite tricky to implement: distinguishing between prefix and postfix increment/decrement operators would require specifying arguments, and allowing it for all types would need a support for pattern matching or optional omission of the type name on methods. All this seems to be an overkill so far.

Tue, May 30, 6:19 AM · Restricted Project
aaron.ballman added a comment to D33531: Clang-tidy readability: avoid const value return.

Would it make sense to silence this diagnostic in the presence of also checking for cert-dcl21-cpp for such operators?

Currently there's no mechanism in clang-tidy to express dependencies or compatibility issues between checks. Moreover, we already have checks that are not meant to be run together, for example, readability-braces-around-statements and its google- incarnation (and other alias checks with different settings). That said, we could whitelist postfix increment and decrement operators in this check. Camillo, WDYT?

Tue, May 30, 5:50 AM · Restricted Project

Mon, May 29

aaron.ballman added a comment to D33531: Clang-tidy readability: avoid const value return.

I'm not opposed to this check, but I'm not keen on having a check that directly contradicts the output from another check. The cert-dcl21-cpp check will diagnose user's code when a postfix operator ++ or -- is *not* const-qualified. Would it make sense to silence this diagnostic in the presence of also checking for cert-dcl21-cpp for such operators?

Mon, May 29, 5:22 AM · Restricted Project

May 25 2017

aaron.ballman closed D33547: Updated getting started guide for visual studio + cmake.

Commit in r303913

May 25 2017, 2:03 PM
aaron.ballman closed D33546: Updated getting started guide for visual studio + cmake.

Commit in r303912

May 25 2017, 2:01 PM
aaron.ballman added a comment to D33546: Updated getting started guide for visual studio + cmake.

It might be overkill, but CMAKE_HOST_SYSTEM_PROCESSOR probably contains what you need. In any case, lgtm even without doing that.

May 25 2017, 1:55 PM
aaron.ballman added a comment to D33470: [clang-tidy] Add misc-default-numerics.

Once you fix the typo in the check, can you run it over some large C++ code bases to see if it finds any results?

May 25 2017, 1:51 PM · Restricted Project
aaron.ballman added a reviewer for D33526: Fix spurious Wunused-lambda-capture warning: rsmith.
May 25 2017, 1:41 PM
aaron.ballman created D33563: Track whether a unary operation can overflow.
May 25 2017, 1:23 PM
aaron.ballman updated the diff for D33546: Updated getting started guide for visual studio + cmake.

Now, with CMake warning. This is only needed for the llvm cmake files (since Clang relies on them anyway).

May 25 2017, 1:18 PM
aaron.ballman added a comment to D33546: Updated getting started guide for visual studio + cmake.

Can we just error out if this is not specified and have CMake print the error message? CMAKE_GENERATOR_TOOLSET contains the value at generation time, and we can just check the value and print an error. Given that you can't successfully build LLVM / clang / etc without it, we might as well fail fast.

We can successfully build LLVM/clang/etc without it (I've done so for *far* too long), but you may get some sporadic "linker out of memory" errors (recompiles will eventually link, however). Also, not everyone may want to use the native toolset. I'm thinking about people who may have something like 8GB of RAM, where the x86 linker is less likely to throttle the machine than the x64 linker.

Since it does work out of the box, why force users to use an extra command line argument?

I've never successfully built clang without using the x64 host toolchain. At the very least, I think we should print a warning.

May 25 2017, 9:03 AM
aaron.ballman added a comment to D33546: Updated getting started guide for visual studio + cmake.

Can we just error out if this is not specified and have CMake print the error message? CMAKE_GENERATOR_TOOLSET contains the value at generation time, and we can just check the value and print an error. Given that you can't successfully build LLVM / clang / etc without it, we might as well fail fast.

May 25 2017, 8:51 AM
aaron.ballman created D33547: Updated getting started guide for visual studio + cmake.
May 25 2017, 5:39 AM
aaron.ballman created D33546: Updated getting started guide for visual studio + cmake.
May 25 2017, 5:38 AM

May 24 2017

aaron.ballman added inline comments to D33470: [clang-tidy] Add misc-default-numerics.
May 24 2017, 10:29 AM · Restricted Project

May 23 2017

aaron.ballman added inline comments to D33398: Mangle __unaligned in Itanium ABI.
May 23 2017, 5:36 AM
aaron.ballman accepted D33430: [clang-tidy] Do not dereference a null BaseType.

LGTM!

May 23 2017, 5:25 AM

May 22 2017

aaron.ballman accepted D31414: [NFC, Refactor] Modernize TemplateIdAnnotation using TrailingObjects.

LGTM!

May 22 2017, 11:08 AM · Restricted Project
aaron.ballman added a comment to D33333: Emit warning when throw exception in destruct or dealloc functions which has a (possible implicit) noexcept specifier.
In D33333#761224, @rnk wrote:

Re: clang-tidy, I would rather implement this as a traditional compiler warning.

In D33333#761126, @jyu2 wrote:

As I said, I don't think checking throw type matching catch handler type is compiler's job. I'd like either silence all warning for that. What do you think?

Consider the following cases:
...
I think the expected behavior in these cases is reasonable (and can be done with a CFG) rather than manually looking at the scope stack like you're doing.

I agree with @jyu2, we should do something simple. I think it would be simple to handle all cases except for h, which requires semantic analysis to figure out if the thrown object would be caught by the catch parameter. Implementing that is more likely to introduce bugs in the compiler than it is to find bugs in user code.

May 22 2017, 11:06 AM
aaron.ballman added a comment to D33333: Emit warning when throw exception in destruct or dealloc functions which has a (possible implicit) noexcept specifier.
In D33333#761126, @jyu2 wrote:
In D33333#760419, @jyu2 wrote:

As an FYI, there is a related check currently under development in clang-tidy; we probably should not duplicate this functionality in both places. See https://reviews.llvm.org/D19201 for the other review.

To my understanding, clang-tidy is kind of source check tool. It does more intensive checking than compiler.
My purpose here is to emit minimum warning form compiler, in case, clang-tidy is not used.

Sort of correct. clang-tidy doesn't necessarily do more intensive checking than the compiler. It's more an AST matching source checking tool for checks that are either expensive or have a higher false-positive rate than we'd want to see in the frontend.

I think that this particular check should probably be in the frontend, like you're doing. My biggest concern is that we do not duplicate functionality between the two tools. D19201 has has a considerable amount of review, so you should look at the test cases there and ensure you are handling them (including the fixits). Hopefully your patch ends up covering all of the functionality in the clang-tidy patch.

Could you add similar tests as the ones that Stanislaw provied in his patch?
Like the one with checking if throw is catched, or the conditional noexcept (by a macro, etc)

Good idea! Could add “marco” test for this. But I am not sure compiler want to add check for throw caught by different catch handler. Because at compile time, compiler can not statically determine which catch handler will be used to caught the exception on all time. I would think that is pragma's responsibility.

For example:

If (a) throw A else throw B;

My main concern there is implicit noexcept gets set by compiler, which cause runtime to termination.

As I said, I don't think checking throw type matching catch handler type is compiler's job. I'd like either silence all warning for that. What do you think?

May 22 2017, 10:42 AM
aaron.ballman added inline comments to D31414: [NFC, Refactor] Modernize TemplateIdAnnotation using TrailingObjects.
May 22 2017, 7:10 AM · Restricted Project
aaron.ballman added inline comments to D33135: [ASTMatchers] Add support for floatLiterals.
May 22 2017, 5:15 AM
aaron.ballman added inline comments to D33094: [ASTMatchers] Add clang-query support for equals matcher.
May 22 2017, 5:05 AM

May 21 2017

aaron.ballman added inline comments to D33094: [ASTMatchers] Add clang-query support for equals matcher.
May 21 2017, 11:12 AM
aaron.ballman added a reviewer for D33333: Emit warning when throw exception in destruct or dealloc functions which has a (possible implicit) noexcept specifier: aaron.ballman.
In D33333#760419, @jyu2 wrote:

As an FYI, there is a related check currently under development in clang-tidy; we probably should not duplicate this functionality in both places. See https://reviews.llvm.org/D19201 for the other review.

To my understanding, clang-tidy is kind of source check tool. It does more intensive checking than compiler.
My purpose here is to emit minimum warning form compiler, in case, clang-tidy is not used.

May 21 2017, 9:33 AM
aaron.ballman added a comment to D33135: [ASTMatchers] Add support for floatLiterals.

By the way, I think that long double is less common than long unsigned literals, so changing unsigned to uint64_t might be something more important?

May 21 2017, 7:48 AM
aaron.ballman added a comment to D19201: [clang-tidy] misc-throw-with-noexcept.

As an FYI, there is a related check being implemented in clang currently; we probably should not duplicate this effort. See https://reviews.llvm.org/D33333.

I think that clang is the right place for this feature, but I am not sure if the patch has all the features (like checking if something will be catched, or not showing warning for conditional noexcepts, which as we have seen
could be a problem for some projects. There also might be some other corner cases that we didn't think about.
Assuming that this patch is ready to land, I would propose to send it to trunk and remove it when the clang's patch will make it to the trunk. I am not sure how much time it will take for other patch to be ready, but maybe we could gather some usefull bug reports in the meantime and also Stanislaw would have a contribution.

May 21 2017, 7:08 AM

May 20 2017

aaron.ballman added a comment to D19201: [clang-tidy] misc-throw-with-noexcept.

As an FYI, there is a related check being implemented in clang currently; we probably should not duplicate this effort. See https://reviews.llvm.org/D33333.

May 20 2017, 5:35 AM
aaron.ballman added a comment to D33333: Emit warning when throw exception in destruct or dealloc functions which has a (possible implicit) noexcept specifier.

As an FYI, there is a related check currently under development in clang-tidy; we probably should not duplicate this functionality in both places. See https://reviews.llvm.org/D19201 for the other review.

May 20 2017, 5:34 AM

May 18 2017

aaron.ballman added a comment to D33304: [clang-tidy][Part1] Add a new module Android and three new checks..

I find the use of "must" at the very least inappropriate. If there was no use case for not including it, it wouldn't be an option. There is also nothing really Android-specific here beside maybe the open64 mess.

On Android, we are requiring this flag. That is why this is part of a new category of Android-specific tidy rules. If you think this belongs more generally in a different category for tidy, can you suggest somewhere else to put it? We didn't want to impose these restrictions for platforms that might not want to be so strict. Also, as with any static analysis, there is the possibility that the original code author intended to "break" the rules, but that is what NOLINT is for.

I'm not keen on putting this in an Android module either, as it's not really Android-specific behavior. For instance, this is also part of a recommended compliant solution for CERT FIO22-C.

I think AOSP has enough specific guidelines and requirements to warrant a separate module (especially, if Android folks have plans to contribute more than one check into it ;). As for this check, if the relevant requirements of CERT and Android are really identical, we could make an alias for the check in the CERT module (or vice versa). Another possibility that comes to mind is to create a new "posix" module specifically for things related to POSIX APIs (or "unix", if we want it to be slightly broader). WDYT?

May 18 2017, 12:17 PM · Restricted Project
aaron.ballman added a comment to D33304: [clang-tidy][Part1] Add a new module Android and three new checks..

I find the use of "must" at the very least inappropriate. If there was no use case for not including it, it wouldn't be an option. There is also nothing really Android-specific here beside maybe the open64 mess.

On Android, we are requiring this flag. That is why this is part of a new category of Android-specific tidy rules. If you think this belongs more generally in a different category for tidy, can you suggest somewhere else to put it? We didn't want to impose these restrictions for platforms that might not want to be so strict. Also, as with any static analysis, there is the possibility that the original code author intended to "break" the rules, but that is what NOLINT is for.

May 18 2017, 10:45 AM · Restricted Project

May 12 2017

aaron.ballman added inline comments to D33135: [ASTMatchers] Add support for floatLiterals.
May 12 2017, 10:08 AM
aaron.ballman added a comment to D32332: Add support for transparent overloadable functions in clang.

I'd be happy with that approach. Do you like it, Aaron?

May 12 2017, 4:47 AM

May 11 2017

aaron.ballman added inline comments to D33094: [ASTMatchers] Add clang-query support for equals matcher.
May 11 2017, 6:41 AM
aaron.ballman accepted D33093: [ASTMatchers] Add support for boolean literals.

LGTM!

May 11 2017, 6:01 AM
aaron.ballman added a reviewer for D33094: [ASTMatchers] Add clang-query support for equals matcher: aaron.ballman.

Please be sure to regenerate the AST matcher documentation as well by running clang/docs/tools/dump_ast_matchers.py

May 11 2017, 5:54 AM

May 10 2017

aaron.ballman added a comment to D32942: [clang-tidy] readability-function-size: add NestingThreshold param..

LGTM!

Thanks.
I guess we want to wait a bit more for @alexfh to leave any comment?

May 10 2017, 2:03 PM · Restricted Project
aaron.ballman accepted D32942: [clang-tidy] readability-function-size: add NestingThreshold param..

LGTM!

May 10 2017, 12:05 PM · Restricted Project
aaron.ballman added inline comments to D32332: Add support for transparent overloadable functions in clang.
May 10 2017, 12:02 PM
aaron.ballman added inline comments to D32942: [clang-tidy] readability-function-size: add NestingThreshold param..
May 10 2017, 8:39 AM · Restricted Project
aaron.ballman accepted D33036: [Support/Compiler.h] - Use gnu::fallthrough for LLVM_FALLTHROUGH when available..

This looks reasonable to me.

May 10 2017, 6:19 AM

May 9 2017

aaron.ballman accepted D32743: [clang-tidy] Add new cert-dcl21-cpp check. .

LGTM!

May 9 2017, 1:37 PM · Restricted Project
aaron.ballman added a comment to D32942: [clang-tidy] readability-function-size: add NestingThreshold param..

Have you tried running this over a large code base to see how frequently the diagnostic triggers? I'm wondering why 10 was chosen as the default threshold.

May 9 2017, 8:11 AM · Restricted Project

May 8 2017

aaron.ballman added inline comments to D32743: [clang-tidy] Add new cert-dcl21-cpp check. .
May 8 2017, 6:53 AM · Restricted Project
aaron.ballman closed D32935: [clang] ThreadSafetyAnalysis.rst: fix spelling of {PT_}GUARDED_VAR attributes.

Committed in r302419, thank you!

May 8 2017, 5:52 AM
aaron.ballman accepted D32935: [clang] ThreadSafetyAnalysis.rst: fix spelling of {PT_}GUARDED_VAR attributes.

LGTM, thank you! Do you need me to commit this for you?

May 8 2017, 5:37 AM
aaron.ballman added inline comments to D32743: [clang-tidy] Add new cert-dcl21-cpp check. .
May 8 2017, 5:33 AM · Restricted Project

May 5 2017

aaron.ballman accepted D32923: [clang-tidy] Use cxxStdInitializerListExpr in modernize-use-emplace.

LGTM!

May 5 2017, 3:48 PM · Restricted Project
aaron.ballman added inline comments to D32743: [clang-tidy] Add new cert-dcl21-cpp check. .
May 5 2017, 1:35 PM · Restricted Project
aaron.ballman closed D32917: [clang] ThreadSafetyAnalysis.rst: do not re-define THREAD_ANNOTATION_ATTRIBUTE__ macro.

Thanks! I've commit in r302275.

May 5 2017, 1:09 PM
aaron.ballman accepted D32917: [clang] ThreadSafetyAnalysis.rst: do not re-define THREAD_ANNOTATION_ATTRIBUTE__ macro.

LGTM!

May 5 2017, 1:05 PM

May 3 2017

aaron.ballman accepted D32767: [clang-tidy] Fix PR32896: detect initializer lists in modernize-use-empalce.

LGTM!

May 3 2017, 12:22 PM · Restricted Project
aaron.ballman accepted D32810: Add cxxStdInitializerListExpr AST matcher.

LGTM!

May 3 2017, 12:00 PM
aaron.ballman added inline comments to D31130: B32239 clang-tidy should not warn about array to pointer decay on system macros.
May 3 2017, 11:43 AM
aaron.ballman added inline comments to D32767: [clang-tidy] Fix PR32896: detect initializer lists in modernize-use-empalce.
May 3 2017, 11:13 AM · Restricted Project
aaron.ballman added a comment to D32743: [clang-tidy] Add new cert-dcl21-cpp check. .

Thank you for working on this check!

May 3 2017, 6:57 AM · Restricted Project