Eugene.Zelenko (Eugene Zelenko)
User

Projects

User does not belong to any projects.

User Details

User Since
Jun 8 2015, 2:10 PM (166 w, 2 d)

Recent Activity

Mon, Aug 13

Eugene.Zelenko added a comment to D48909: [clang-doc] Update BitcodeReader to use llvm::Error.

You could use Differential revision: <review URL> in commit description to close review automatically.

Mon, Aug 13, 3:40 PM · Restricted Project

Fri, Aug 10

Eugene.Zelenko added reviewers for D50580: [clang-tidy] Abseil: no namespace check: aaron.ballman, ilya-biryukov.
Fri, Aug 10, 4:55 PM · Restricted Project
Eugene.Zelenko added a comment to D50580: [clang-tidy] Abseil: no namespace check.

Check documentation is missing.

Fri, Aug 10, 4:54 PM · Restricted Project
Eugene.Zelenko added inline comments to D50542: [clang-tidy] Add abseil-no-internal-deps check.
Fri, Aug 10, 10:21 AM · Restricted Project

Thu, Aug 9

Eugene.Zelenko retitled D50542: [clang-tidy] Add abseil-no-internal-deps check from [Clang-Tidy] Check for abseil to [clang-tidy] Add abseil-no-internal-deps check.
Thu, Aug 9, 7:05 PM · Restricted Project
Eugene.Zelenko added a comment to D50542: [clang-tidy] Add abseil-no-internal-deps check.

Why not to use dependencies instead of deps in check's name?

Thu, Aug 9, 7:04 PM · Restricted Project

Tue, Aug 7

Eugene.Zelenko edited reviewers for D50389: [clang-tidy] Abseil: integral division of Duration check, added: alexfh, ilya-biryukov; removed: Restricted Project.
Tue, Aug 7, 10:22 AM · Restricted Project

Sat, Aug 4

Eugene.Zelenko added inline comments to D50299: Migrate to std::function instead of static member functions for callbacks.
Sat, Aug 4, 9:59 PM

Mon, Jul 30

Eugene.Zelenko added inline comments to D50034: Add a CommandGuide for llvm-objdump.
Mon, Jul 30, 7:23 PM

Fri, Jul 27

Eugene.Zelenko added reviewers for D49910: [clang-tidy] Recognize [[clang::reinitializes]] attribute in bugprone-use-after-move: alexfh, hokein.
Fri, Jul 27, 10:13 AM · Restricted Project
Eugene.Zelenko added reviewers for D49918: [clang-tidy] Sequence init statements, declarations, and conditions correctly in if, switch, and while: alexfh, aaron.ballman, hokein.
Fri, Jul 27, 10:12 AM · Restricted Project

Thu, Jul 26

Eugene.Zelenko added inline comments to D49890: Clang-Tidy Export Problem.
Thu, Jul 26, 6:40 PM · Restricted Project
Eugene.Zelenko added inline comments to D49890: Clang-Tidy Export Problem.
Thu, Jul 26, 5:52 PM · Restricted Project
Eugene.Zelenko added reviewers for D49890: Clang-Tidy Export Problem: aaron.ballman, hokein, ilya-biryukov.
Thu, Jul 26, 5:51 PM · Restricted Project
Eugene.Zelenko added inline comments to D49890: Clang-Tidy Export Problem.
Thu, Jul 26, 5:50 PM · Restricted Project
Eugene.Zelenko retitled D49864: [clang-tidy] The script clang-tidy-diff.py doesn't accept 'pass by' options (--) from The script clang-tidy-diff.py doesn't accept 'pass by' options (--) to [clang-tidy] The script clang-tidy-diff.py doesn't accept 'pass by' options (--).
Thu, Jul 26, 10:13 AM · Restricted Project
Eugene.Zelenko removed a reviewer for D49223: [AST] Check described template at structural equivalence check.: Eugene.Zelenko.
Thu, Jul 26, 10:04 AM · Restricted Project
Eugene.Zelenko added reviewers for D49851: [clang-tidy] run-clang-tidy add synchronisation to the output: alexfh, hokein.
Thu, Jul 26, 9:00 AM · Restricted Project

Tue, Jul 24

Eugene.Zelenko added inline comments to D49114: [clang-tidy] Add a check for "magic numbers".
Tue, Jul 24, 7:11 PM
Eugene.Zelenko added inline comments to D49114: [clang-tidy] Add a check for "magic numbers".
Tue, Jul 24, 7:00 PM

Tue, Jul 17

Eugene.Zelenko added inline comments to D49114: [clang-tidy] Add a check for "magic numbers".
Tue, Jul 17, 4:37 PM
Eugene.Zelenko added a comment to D49114: [clang-tidy] Add a check for "magic numbers".

Adding C++ Core Guidelines alias is definitely low-hanging fruit which could be implemented within this patch.

Tue, Jul 17, 10:18 AM
Eugene.Zelenko added a comment to D45444: [clang-tidy] implement new check for const-correctness.

But you could also take a look at the enforcement of some coding
guidelines (my personal todolists:
https://github.com/JonasToth/CppCoreGuidelinesTooling 
https://github.com/JonasToth/HighIntegrityTooling). There are always low
hanging fruit checks. Implementing something there helps getting used to
the codebase of clang, too.

Tue, Jul 17, 10:14 AM
Eugene.Zelenko removed a reviewer for D49412: Enrich inline messages: Eugene.Zelenko.
Tue, Jul 17, 10:11 AM

Jul 14 2018

Eugene.Zelenko retitled D49334: [LLDB} Added syntax highlighting support from Added syntax highlighting support to [LLDB} Added syntax highlighting support.
Jul 14 2018, 10:46 AM · Restricted Project

Jul 11 2018

Eugene.Zelenko added inline comments to D49114: [clang-tidy] Add a check for "magic numbers".
Jul 11 2018, 4:45 PM
Eugene.Zelenko added inline comments to D49114: [clang-tidy] Add a check for "magic numbers".
Jul 11 2018, 4:08 PM
Eugene.Zelenko added a comment to D49114: [clang-tidy] Add a check for "magic numbers".

C++ Core Guidelines contains ES.45: Avoid "magic constants"; use symbolic constants, so I think check should be moved into cppcoreguidelines module.

The same guidelines existed in medical and aerospace environments for many years before C++ Core Guidelines. In fact the same checker should run just fine on C code (I'm not sure about Objective-C).

I am not opposed to moving it, if this is where the consensus leads - I know where to find it :)

Jul 11 2018, 3:20 PM
Eugene.Zelenko added a comment to D49114: [clang-tidy] Add a check for "magic numbers".

C++ Core Guidelines contains ES.45: Avoid "magic constants"; use symbolic constants, so I think check should be moved into cppcoreguidelines module.

Jul 11 2018, 1:33 PM

Jul 10 2018

Eugene.Zelenko added a comment to D49114: [clang-tidy] Add a check for "magic numbers".

It's highly likely that this part of coding guidelines.

Jul 10 2018, 2:06 PM

Jul 3 2018

Eugene.Zelenko added inline comments to D48908: [clang-doc] Pass over function-internal declarations.
Jul 3 2018, 7:29 PM · Restricted Project
Eugene.Zelenko added inline comments to D48909: [clang-doc] Update BitcodeReader to use llvm::Error.
Jul 3 2018, 7:28 PM · Restricted Project
Eugene.Zelenko removed a reviewer for D48894: [AST] Rename some Redeclarable functions to reduce confusion: Eugene.Zelenko.
Jul 3 2018, 3:54 PM · Restricted Project
Eugene.Zelenko added a comment to D48866: [clang-tidy] Add incorrect-pointer-cast checker.

bugprone seems to be proper category for this check.

Jul 3 2018, 8:16 AM

Jun 20 2018

Eugene.Zelenko added inline comments to D43424: [clang-doc] Implement a (simple) Markdown generator.
Jun 20 2018, 1:44 PM · Restricted Project

May 23 2018

Eugene.Zelenko added a comment to D47157: Warning for framework headers using double quote includes.

Hi Eugene,

You could just not include new warning into -Wall and -Wextra. Those who will want to check #include statements correctness could enable it explicitly.

This is exactly what's happening in the patch, the new warning isn't part of -Wall or -Wextra, and is marked DefaultIgnore, which means that will fire only when -Wquoted-include-in-framework-header is passed to the driver. Am I missing something from your explanation?

Thanks,

May 23 2018, 5:05 PM

May 21 2018

Eugene.Zelenko added a comment to D47157: Warning for framework headers using double quote includes.

The warning is off by default.

We typically do not add off-by-default warnings because experience has shown the rarely get enabled in practice. Can you explain a bit more about why this one is off by default?

Right. I believe this is going to be used in practice, the reason I'm adding it involves some user demand for such warning. Such quoted include use in frameworks happen often and we would like a smooth transition to happen here (e.g. do not initially affect -Werror users). If it proves worth it, we can migrate to on by default in the future. It wouldn't be a problem if we have it on by default on open source and disable by default downstream, but I rather be consistent.

May 21 2018, 3:49 PM
Eugene.Zelenko added a comment to D47157: Warning for framework headers using double quote includes.

See also PR22165.

May 21 2018, 2:02 PM

May 8 2018

Eugene.Zelenko added a comment to D46602: [clang-tidy] Store checks profiling info as JSON files.

I think will be good idea to store data in JSON format too.

May 8 2018, 3:30 PM · Restricted Project
Eugene.Zelenko added inline comments to D43778: [clang-tidy] Adding RestrictIncludes check to Fuchsia module.
May 8 2018, 2:30 PM · Restricted Project

May 6 2018

Eugene.Zelenko added inline comments to D46504: [clang-tidy] Profile is a per-AST (per-TU) data..
May 6 2018, 4:20 PM · Restricted Project

May 3 2018

Eugene.Zelenko added a comment to D46407: [tools] Introduce llvm-strip.

Will be good idea to add documentation and item in Release Notes.

May 3 2018, 3:41 PM
Eugene.Zelenko added a comment to D46378: Undef the macros after their use.

I don't see anything wrong with this change, but will be good idea to wait for code owner approval.

May 3 2018, 6:11 AM

May 2 2018

Eugene.Zelenko added inline comments to D45702: [clang-tidy] Add a new check, readability-simplify-subscript-expr, that simplifies subscript expressions..
May 2 2018, 6:24 AM · Restricted Project

May 1 2018

Eugene.Zelenko added reviewers for D46317: [clang-tidy] New check bugprone-map-subscript-operator-lookup: hokein, aaron.ballman, ilya-biryukov.
May 1 2018, 10:12 AM · Restricted Project
Eugene.Zelenko added inline comments to D46317: [clang-tidy] New check bugprone-map-subscript-operator-lookup.
May 1 2018, 10:12 AM · Restricted Project
Eugene.Zelenko added inline comments to D46243: Move Schedule class to header file for allowing inheritance.
May 1 2018, 7:34 AM

Apr 30 2018

Eugene.Zelenko removed a reviewer for D46265: StackColoring: better handling of statically unreachable code: Eugene.Zelenko.
Apr 30 2018, 8:26 AM
Eugene.Zelenko added inline comments to D46243: Move Schedule class to header file for allowing inheritance.
Apr 30 2018, 6:36 AM
Eugene.Zelenko removed a reviewer for D46243: Move Schedule class to header file for allowing inheritance: Eugene.Zelenko.
Apr 30 2018, 6:36 AM

Apr 22 2018

Eugene.Zelenko edited reviewers for D45932: [clang-tidy][modernize-raw-string-literal] Don't replace upper ASCII with raw literals, added: alexfh, aaron.ballman, hokein, ilya-biryukov; removed: xazax.hun, LegalizeAdulthood.
Apr 22 2018, 4:23 PM · Restricted Project

Apr 20 2018

Eugene.Zelenko added reviewers for D45891: [clang-tidy] Improve bugprone-unused-return-value check: hokein, aaron.ballman, ilya-biryukov.
Apr 20 2018, 1:23 PM · Restricted Project

Apr 19 2018

Eugene.Zelenko added inline comments to D45702: [clang-tidy] Add a new check, readability-simplify-subscript-expr, that simplifies subscript expressions..
Apr 19 2018, 6:30 PM · Restricted Project
Eugene.Zelenko added inline comments to D45702: [clang-tidy] Add a new check, readability-simplify-subscript-expr, that simplifies subscript expressions..
Apr 19 2018, 6:00 PM · Restricted Project

Apr 18 2018

Eugene.Zelenko added a comment to D45679: [clang-tidy] Add ExprMutationAnalyzer, that analyzes whether an expression is mutated within a statement..

You are doing a great job and i learn new stuff :)

  • What do you think about having these functions in a class? Now, we need to recalculate and reanalyze the scope for every variable, multiple times (reference tracking). It would be nice to do it as lazy as possible and memorize the results. Especially addressing the use-case for the const-check, storing that a reference is not modified will save a lot of work = performance
Apr 18 2018, 10:32 AM · Restricted Project

Apr 17 2018

Eugene.Zelenko added a comment to D45679: [clang-tidy] Add ExprMutationAnalyzer, that analyzes whether an expression is mutated within a statement..

*hust* /llvm/tools/clang/lib/Analysis/PseudoConstantAnalysis.cpp

I will check this one first, before we get crazy and implemented something twice.

Apr 17 2018, 6:57 AM · Restricted Project

Apr 16 2018

Eugene.Zelenko added inline comments to D45702: [clang-tidy] Add a new check, readability-simplify-subscript-expr, that simplifies subscript expressions..
Apr 16 2018, 3:47 PM · Restricted Project
Eugene.Zelenko added reviewers for D45702: [clang-tidy] Add a new check, readability-simplify-subscript-expr, that simplifies subscript expressions.: alexfh, hokein, aaron.ballman, ilya-biryukov.
Apr 16 2018, 3:40 PM · Restricted Project
Eugene.Zelenko added a comment to D45702: [clang-tidy] Add a new check, readability-simplify-subscript-expr, that simplifies subscript expressions..

Please also take a look on 26817 for another idea for .data().

Apr 16 2018, 3:39 PM · Restricted Project
Eugene.Zelenko added a comment to D45444: [clang-tidy] implement new check for const-correctness.
  • I would imagine things could get messier if this check expands to also check for turning member functions const: it's basically checking CxxThisExpr, being a handle, is not modified within a member function, but note there's no VarDecl for "this".
Apr 16 2018, 10:35 AM

Apr 15 2018

Eugene.Zelenko added a comment to D45679: [clang-tidy] Add ExprMutationAnalyzer, that analyzes whether an expression is mutated within a statement..

See also D45444.

Apr 15 2018, 7:22 PM · Restricted Project
Eugene.Zelenko retitled D45679: [clang-tidy] Add ExprMutationAnalyzer, that analyzes whether an expression is mutated within a statement. from Add a new check, readability-unmodified-non-const-variable, that finds declarations of non-const variables that never get modified. to [clang-tidy] Add a new check, readability-unmodified-non-const-variable, that finds declarations of non-const variables that never get modified..
Apr 15 2018, 7:21 PM · Restricted Project

Apr 12 2018

Eugene.Zelenko added a comment to D45601: Warn on bool* to bool conversion.

There is Clang-tidy's readability-implicit-bool-conversion check.

Apr 12 2018, 4:39 PM
Eugene.Zelenko removed a reviewer for D45420: [NFC] [LoopUtil] Moved RecurrenceDescriptor/LoopDescriptor from Transform/Utils/LoopUtils.* to Analysis tree: Eugene.Zelenko.
Apr 12 2018, 6:34 AM
Eugene.Zelenko added inline comments to D45420: [NFC] [LoopUtil] Moved RecurrenceDescriptor/LoopDescriptor from Transform/Utils/LoopUtils.* to Analysis tree.
Apr 12 2018, 6:31 AM

Apr 10 2018

Eugene.Zelenko added a comment to D45392: [clang-tidy] add new check to find out objc ivars which do not have prefix '_'.

How about doing same for objc-property-declaration?

Apr 10 2018, 6:07 PM · Restricted Project
Eugene.Zelenko added a comment to D45444: [clang-tidy] implement new check for const-correctness.

It'll be good idea to have option to apply this check for pointer/references only, or include built-in types/enums.

Agreed. I aim at a mark handles const(default on), mark values const(default on for objects), mark pointer (const int * >> const <<) const (default off)

What do you mean by built-in types/enums? That they are configurable separate to objects?

Apr 10 2018, 10:32 AM
Eugene.Zelenko added inline comments to D45468: [clang-tidy] Adding Fuchsia checker for human-readable logging.
Apr 10 2018, 10:19 AM · Restricted Project

Apr 9 2018

Eugene.Zelenko added a comment to D45444: [clang-tidy] implement new check for const-correctness.

Will be good idea to add HICPP alias.

Apr 9 2018, 3:16 PM
Eugene.Zelenko added a comment to D45444: [clang-tidy] implement new check for const-correctness.

It'll be good idea to have option to apply this check for pointer/references only, or include built-in types/enums.

Apr 9 2018, 11:01 AM

Apr 8 2018

Eugene.Zelenko added a comment to D45392: [clang-tidy] add new check to find out objc ivars which do not have prefix '_'.

If this is Apple guideline, check name should reflect this. I think will be good idea to have general check for Apple naming conventions instead of separate checks for specific situations like objc-ivar-declaration and objc-property-declaration.

Thanks for the suggestion. I understand your point that they are both naming convention, however, they are about different components and using totally different naming rules. PropertyDeclarationCheck is already a very complicated check (the most complicated one for ObjC), I would rather not make it more heavy and try my best to split independent logic to different checks.

See readability-identifier-naming as example of multiple rules in one check.

I took a look at IdentifierNamingCheck. Here's my thought:

  1. IdentifierNamingCheck is trying to apply configurable naming convention to C++ identifiers, and all the identifiers will share the same style set. That is not the case of ObjC, where we follow Apple's programming guide, and different types of identifiers are using different style.
  2. Such pattern can handle complicated requirements but to me it is not simple enough to read and maintain. I would rather keep things simple and clear as long as we have choice.

    However, this check provides a good example of refactoring if in the future we have the needs of organizing complicated naming styles. Moving from simplicity to complexity is always easier. Thanks for pointing this out for us.

My point is not flexibility of configuration, but handling of various types of identifiers in same check, even if conventions are different.

Yes I understand but I mean "flexibility of configuration" is one of the reasons of handling of various types of identifiers in same check, but we don't need it here.

Apr 8 2018, 3:22 PM · Restricted Project
Eugene.Zelenko added a comment to D45392: [clang-tidy] add new check to find out objc ivars which do not have prefix '_'.

If this is Apple guideline, check name should reflect this. I think will be good idea to have general check for Apple naming conventions instead of separate checks for specific situations like objc-ivar-declaration and objc-property-declaration.

Thanks for the suggestion. I understand your point that they are both naming convention, however, they are about different components and using totally different naming rules. PropertyDeclarationCheck is already a very complicated check (the most complicated one for ObjC), I would rather not make it more heavy and try my best to split independent logic to different checks.

See readability-identifier-naming as example of multiple rules in one check.

I took a look at IdentifierNamingCheck. Here's my thought:

  1. IdentifierNamingCheck is trying to apply configurable naming convention to C++ identifiers, and all the identifiers will share the same style set. That is not the case of ObjC, where we follow Apple's programming guide, and different types of identifiers are using different style.
  2. Such pattern can handle complicated requirements but to me it is not simple enough to read and maintain. I would rather keep things simple and clear as long as we have choice.

    However, this check provides a good example of refactoring if in the future we have the needs of organizing complicated naming styles. Moving from simplicity to complexity is always easier. Thanks for pointing this out for us.
Apr 8 2018, 7:05 AM · Restricted Project

Apr 7 2018

Eugene.Zelenko added a comment to D45392: [clang-tidy] add new check to find out objc ivars which do not have prefix '_'.

If this is Apple guideline, check name should reflect this. I think will be good idea to have general check for Apple naming conventions instead of separate checks for specific situations like objc-ivar-declaration and objc-property-declaration.

Thanks for the suggestion. I understand your point that they are both naming convention, however, they are about different components and using totally different naming rules. PropertyDeclarationCheck is already a very complicated check (the most complicated one for ObjC), I would rather not make it more heavy and try my best to split independent logic to different checks.

Apr 7 2018, 8:22 PM · Restricted Project

Apr 6 2018

Eugene.Zelenko added a comment to D45392: [clang-tidy] add new check to find out objc ivars which do not have prefix '_'.

If this is Apple guideline, check name should reflect this. I think will be good idea to have general check for Apple naming conventions instead of separate checks for specific situations like objc-ivar-declaration and objc-property-declaration.

Apr 6 2018, 5:23 PM · Restricted Project
Eugene.Zelenko retitled D45392: [clang-tidy] add new check to find out objc ivars which do not have prefix '_' from add new check to find out objc ivars which do not have prefix '_' to [clang-tidy] add new check to find out objc ivars which do not have prefix '_'.
Apr 6 2018, 5:16 PM · Restricted Project

Apr 4 2018

Eugene.Zelenko added a comment to D44519: Add llvm-exegesis tool..

I found /usr/lib/libpfm.so.3.10.0

Apr 4 2018, 11:34 AM
Eugene.Zelenko added a comment to D44519: Add llvm-exegesis tool..

Looks like CMake is not checking for external dependencies properly. I got build error on RHEL 6 because of missing perfmon/perf_event.h. Most likely perfmon version is too old on RHEL6.

Apr 4 2018, 11:22 AM

Mar 31 2018

Eugene.Zelenko added inline comments to D37014: [clang-tidy] Add a checker to remove useless intermediate variables before return statements with comparisons.
Mar 31 2018, 8:58 AM · Restricted Project

Mar 29 2018

Eugene.Zelenko added inline comments to D45050: [clang-tidy] New checker for not null-terminated result caused by strlen or wcslen.
Mar 29 2018, 2:49 PM · Restricted Project
Eugene.Zelenko added inline comments to D45050: [clang-tidy] New checker for not null-terminated result caused by strlen or wcslen.
Mar 29 2018, 2:11 PM · Restricted Project
Eugene.Zelenko added a comment to D45059: [clang-tidy] Add check to catch comparisons in TEMP_FAILURE_RETRY.

Will be good idea to clarify where TEMP_FAILURE_RETRY come from.

Mar 29 2018, 1:21 PM · Restricted Project
Eugene.Zelenko retitled D45059: [clang-tidy] Add check to catch comparisons in TEMP_FAILURE_RETRY from Add a clang-tidy check to catch comparisons in TEMP_FAILURE_RETRY to [clang-tidy] Add check to catch comparisons in TEMP_FAILURE_RETRY.
Mar 29 2018, 1:01 PM · Restricted Project
Eugene.Zelenko added inline comments to D45059: [clang-tidy] Add check to catch comparisons in TEMP_FAILURE_RETRY.
Mar 29 2018, 1:01 PM · Restricted Project
Eugene.Zelenko added inline comments to D45050: [clang-tidy] New checker for not null-terminated result caused by strlen or wcslen.
Mar 29 2018, 11:10 AM · Restricted Project
Eugene.Zelenko edited reviewers for D45050: [clang-tidy] New checker for not null-terminated result caused by strlen or wcslen, added: hokein, ilya-biryukov; removed: xazax.hun.
Mar 29 2018, 11:10 AM · Restricted Project
Eugene.Zelenko added inline comments to D45050: [clang-tidy] New checker for not null-terminated result caused by strlen or wcslen.
Mar 29 2018, 10:49 AM · Restricted Project
Eugene.Zelenko added inline comments to D37014: [clang-tidy] Add a checker to remove useless intermediate variables before return statements with comparisons.
Mar 29 2018, 10:23 AM · Restricted Project

Mar 28 2018

Eugene.Zelenko added inline comments to D33537: [clang-tidy] Exception Escape Checker.
Mar 28 2018, 10:06 AM · Restricted Project

Mar 27 2018

Eugene.Zelenko added a comment to D44948: Add diagnostic -Waggregate-ctors, "aggregate type has user-declared constructors".

Please mention new diagnostics in documentation and Release Notes.

Mar 27 2018, 4:46 PM · Restricted Project
Eugene.Zelenko added reviewers for D44948: Add diagnostic -Waggregate-ctors, "aggregate type has user-declared constructors": rsmith, erichkeane, dblaikie.
Mar 27 2018, 4:45 PM · Restricted Project

Mar 26 2018

Eugene.Zelenko added a comment to D43341: [clang-doc] Implement reducer portion of the frontend framework.

Please mention new tool in Release Notes and use :doc: to refer to its manual.

Mar 26 2018, 5:02 PM · Restricted Project
Eugene.Zelenko added a comment to D44912: [clang-doc] Removing -Wunused-variable warning.

I think it's fine to commit trivial fixes without going via review.

Mar 26 2018, 4:07 PM · Restricted Project
Eugene.Zelenko added reviewers for D44906: [clang-tidy] Define __clang_analyzer__ macro for clang-tidy for compatibility with clang static analyzer: alexfh, hokein, aaron.ballman.
Mar 26 2018, 1:37 PM · Restricted Project
Eugene.Zelenko added a comment to D41648: [clang-tidy] implement cppcoreguidelines macro rules.

Which checks do you have in mind?

Mar 26 2018, 10:10 AM
Eugene.Zelenko added inline comments to D41648: [clang-tidy] implement cppcoreguidelines macro rules.
Mar 26 2018, 9:07 AM
Eugene.Zelenko added inline comments to D41648: [clang-tidy] implement cppcoreguidelines macro rules.
Mar 26 2018, 9:04 AM

Mar 23 2018

Eugene.Zelenko added a comment to D44826: Add -Wunused-using, a warning that finds unused using declarations..

Please also mention new warning in Release Notes and documentation.

Mar 23 2018, 11:22 AM · Restricted Project
Eugene.Zelenko added a comment to D44826: Add -Wunused-using, a warning that finds unused using declarations..

This duplicates Clang-tidy misc-unused-using-decls. If Clang will provide same or better functionality, it should be removed.

Mar 23 2018, 11:19 AM · Restricted Project
Eugene.Zelenko added inline comments to D37014: [clang-tidy] Add a checker to remove useless intermediate variables before return statements with comparisons.
Mar 23 2018, 8:07 AM · Restricted Project
Eugene.Zelenko added inline comments to D33537: [clang-tidy] Exception Escape Checker.
Mar 23 2018, 6:55 AM · Restricted Project