whisperity (Whisperity)
CodeChecker-CodeCompass Development

Projects

User does not belong to any projects.

User Details

User Since
Dec 6 2016, 10:52 AM (83 w, 6 d)

Recent Activity

Thu, Jul 5

whisperity added inline comments to D32859: [Analyzer] Iterator Checker - Part 5: Move Assignment of Containers.
Thu, Jul 5, 5:07 AM
whisperity added inline comments to D32845: [Analyzer] Iterator Checker - Part 4: Mismatched iterator checker for function parameters.
Thu, Jul 5, 3:13 AM
whisperity added inline comments to D32747: [Analyzer] Iterator Checker - Part 3: Invalidation check, first for (copy) assignments.
Thu, Jul 5, 2:43 AM

Mon, Jul 2

whisperity added a comment to D45094: [LibTooling] Make interface of VFS injection into ClangTool more user-friendly.

Pinging this as the talk has stalled.

Mon, Jul 2, 12:30 AM · Restricted Project

Jun 16 2018

whisperity accepted D45532: [StaticAnalyzer] Checker to find uninitialized fields after a constructor call.

Ah, and the function names in the test files have been made more logical.

Jun 16 2018, 1:41 PM

Jun 3 2018

whisperity requested changes to D45050: [clang-tidy] New checker for not null-terminated result caused by strlen or wcslen.

In general, make sure the documentation page renders well in a browser.

Jun 3 2018, 7:45 AM · Restricted Project

May 16 2018

whisperity added a dependent revision for D46891: [StaticAnalyzer] Added a getLValue method to ProgramState for bases: D45532: [StaticAnalyzer] Checker to find uninitialized fields after a constructor call.
May 16 2018, 12:18 AM
whisperity added a dependency for D45532: [StaticAnalyzer] Checker to find uninitialized fields after a constructor call: D46891: [StaticAnalyzer] Added a getLValue method to ProgramState for bases.
May 16 2018, 12:18 AM

Apr 29 2018

whisperity added a comment to D45532: [StaticAnalyzer] Checker to find uninitialized fields after a constructor call.

Two minor comments.

Apr 29 2018, 3:25 AM

Apr 26 2018

whisperity added a comment to D46081: [analyzer] Expand conversion check to check more expressions for overflow and underflow.

While I understand extending the analyzer to cover more is a good approach, there is -Wconversion which seemingly covers this -- or at least the trivial case(?):

Apr 26 2018, 2:41 AM · Restricted Project
whisperity updated subscribers of D46081: [analyzer] Expand conversion check to check more expressions for overflow and underflow.
Apr 26 2018, 2:36 AM · Restricted Project

Apr 23 2018

whisperity added a comment to D45532: [StaticAnalyzer] Checker to find uninitialized fields after a constructor call.

@george.karpenkov @NoQ bugprone. as a category sounds nice. It also nicely corresponds to the Clang-Tidy bugprone- category. It would not be nice to further fragment the "top levels" of checker categories.

Apr 23 2018, 1:55 AM

Apr 19 2018

whisperity added inline comments to D45095: [clang-tidy] Align usage of ClangTool interface with new VFS injection.
Apr 19 2018, 7:22 AM · Restricted Project

Apr 17 2018

whisperity requested changes to D45532: [StaticAnalyzer] Checker to find uninitialized fields after a constructor call.

There is something that came up in my mind:

Apr 17 2018, 3:19 AM
whisperity added a comment to D45532: [StaticAnalyzer] Checker to find uninitialized fields after a constructor call.

This bug report also mentions assignment operator. But for that a warning may be not so useful. In that case the members of the assigned to object should have some initialized value already which the programmer may not want to overwrite in the assignment operator.

I believe there's a checker for that already, but I'm really not sure whether UndefinedAssignmentChecker covers all such cases.

Apr 17 2018, 3:17 AM

Apr 16 2018

whisperity added a comment to D45407: [StaticAnalyzer] Added notes to the plist output.

@NoQ The problem with emitting notes as events is that we lose the information that the node was a note. How does Xcode behave with these notes? Does it ignore them, or can read them from the command-line output of the analyser?

Apr 16 2018, 6:50 AM · Restricted Project

Apr 12 2018

whisperity requested changes to D45532: [StaticAnalyzer] Checker to find uninitialized fields after a constructor call.

Sorry, one comment has gone missing meanwhile, I'm still getting used to this interface and hit Submit early.

Apr 12 2018, 2:29 AM
whisperity added a comment to D45532: [StaticAnalyzer] Checker to find uninitialized fields after a constructor call.

@NoQ Do you reckon these tests files are too long? Perhaps the one about this inheritance, that inheritance, diamond inheritance, etc. could be split into multiple files.

Apr 12 2018, 2:20 AM

Apr 9 2018

whisperity added inline comments to D45095: [clang-tidy] Align usage of ClangTool interface with new VFS injection.
Apr 9 2018, 5:52 AM · Restricted Project
whisperity added inline comments to D45094: [LibTooling] Make interface of VFS injection into ClangTool more user-friendly.
Apr 9 2018, 5:45 AM · Restricted Project

Apr 6 2018

whisperity updated the diff for D45096: Allow the creation of human-friendly ASTDumper to arbitrary output stream.

Added comments on what nullptr means at call sites.

Apr 6 2018, 4:29 AM · Restricted Project
whisperity added a comment to D45096: Allow the creation of human-friendly ASTDumper to arbitrary output stream.

It is also std-out (llvm::outs()) in case of nullptr and not std-err.

Apr 6 2018, 4:29 AM · Restricted Project
whisperity updated the summary of D45096: Allow the creation of human-friendly ASTDumper to arbitrary output stream.
Apr 6 2018, 4:22 AM · Restricted Project

Apr 5 2018

whisperity updated the diff for D45095: [clang-tidy] Align usage of ClangTool interface with new VFS injection.

Update to be in line with contents in dependency patch.

Apr 5 2018, 1:09 PM · Restricted Project
whisperity updated the diff for D45094: [LibTooling] Make interface of VFS injection into ClangTool more user-friendly.

Simplify the patch.

Apr 5 2018, 1:08 PM · Restricted Project
whisperity updated the diff for D45094: [LibTooling] Make interface of VFS injection into ClangTool more user-friendly.
  • Use an even more explicit way with the documentation requiring that the file system should be an overlay.
  • Add a method to easily overlay a FileSystem above the real one.
Apr 5 2018, 1:03 PM · Restricted Project
whisperity updated the summary of D45094: [LibTooling] Make interface of VFS injection into ClangTool more user-friendly.
Apr 5 2018, 1:03 PM · Restricted Project
whisperity added a comment to D45096: Allow the creation of human-friendly ASTDumper to arbitrary output stream.

@alexfh I have updated the patch. I don't have commit rights, so if you think this is good to go, could you please commit for me?

Apr 5 2018, 12:45 PM · Restricted Project
whisperity requested review of D45096: Allow the creation of human-friendly ASTDumper to arbitrary output stream.
Apr 5 2018, 12:10 PM · Restricted Project
whisperity updated the diff for D45096: Allow the creation of human-friendly ASTDumper to arbitrary output stream.
  • Overload removed, now only one CreateASTDumper function remains.
  • Updated the call sites of this function to use this call.
Apr 5 2018, 12:10 PM · Restricted Project
whisperity planned changes to D45095: [clang-tidy] Align usage of ClangTool interface with new VFS injection.
Apr 5 2018, 11:54 AM · Restricted Project

Mar 30 2018

whisperity created D45096: Allow the creation of human-friendly ASTDumper to arbitrary output stream.
Mar 30 2018, 8:57 AM · Restricted Project
whisperity updated subscribers of D45094: [LibTooling] Make interface of VFS injection into ClangTool more user-friendly.
Mar 30 2018, 8:40 AM · Restricted Project
whisperity added a dependent revision for D45094: [LibTooling] Make interface of VFS injection into ClangTool more user-friendly: D45095: [clang-tidy] Align usage of ClangTool interface with new VFS injection.
Mar 30 2018, 8:39 AM · Restricted Project
whisperity added a dependency for D45095: [clang-tidy] Align usage of ClangTool interface with new VFS injection: D45094: [LibTooling] Make interface of VFS injection into ClangTool more user-friendly.
Mar 30 2018, 8:39 AM · Restricted Project
whisperity created D45095: [clang-tidy] Align usage of ClangTool interface with new VFS injection.
Mar 30 2018, 8:39 AM · Restricted Project
whisperity created D45094: [LibTooling] Make interface of VFS injection into ClangTool more user-friendly.
Mar 30 2018, 8:36 AM · Restricted Project

Feb 28 2018

whisperity awarded D30691: [analyzer] Support for naive cross translational unit analysis a Party Time token.
Feb 28 2018, 5:40 AM

Feb 9 2018

whisperity resigned from D43120: [clang-tidy] New checker for exceptions that are created but not thrown.

Works for me but I haven't any sayings in these. 😇

Feb 9 2018, 6:23 AM · Restricted Project
whisperity requested changes to D43120: [clang-tidy] New checker for exceptions that are created but not thrown.
Feb 9 2018, 5:29 AM · Restricted Project
whisperity updated subscribers of D43120: [clang-tidy] New checker for exceptions that are created but not thrown.
Feb 9 2018, 5:15 AM · Restricted Project

Nov 24 2017

whisperity updated subscribers of D37437: [analyzer] Fix some checker's output plist not containing the checker name.
Nov 24 2017, 3:14 AM

Aug 31 2017

whisperity added a comment to D30691: [analyzer] Support for naive cross translational unit analysis.

The Python code here still uses mangled name in their wording. Does this mean this patch is yet to be updated with the USR management in the parent patch?

Aug 31 2017, 4:19 AM

Aug 10 2017

whisperity added inline comments to D34512: Add preliminary Cross Translation Unit support library.
Aug 10 2017, 3:01 AM

Aug 9 2017

whisperity added a comment to D34512: Add preliminary Cross Translation Unit support library.

Apart from those in the in-line comments, I have a question: how safe is this library to Release builds? I know this is only a submodule dependency for the "real deal" in D30691, but I have seen some asserts that "imported function should already have a body" and such.

Aug 9 2017, 9:20 AM

Jul 31 2017

whisperity updated subscribers of D33722: [clang-tidy] Add checker for undelegated copy of base classes.
Jul 31 2017, 9:22 AM · Restricted Project

Jul 24 2017

whisperity updated subscribers of D30691: [analyzer] Support for naive cross translational unit analysis.
Jul 24 2017, 7:48 AM

Jul 20 2017

whisperity updated subscribers of D35687: [StaticAnalyzer] Skip simulating branches where for-loops take 0 unrolled step.
Jul 20 2017, 9:07 AM
whisperity updated subscribers of D35684: [StaticAnalyzer] LoopUnrolling: Track a LoopStack in order to completely unroll specific loops.
Jul 20 2017, 8:13 AM
whisperity updated subscribers of D35668: [CFG] Add LoopExit information to CFG.
Jul 20 2017, 8:13 AM
whisperity updated subscribers of D35670: [StaticAnalyzer] Handle LoopExit CFGElement in the analyzer.
Jul 20 2017, 8:13 AM

Jul 14 2017

whisperity added inline comments to D32700: [clang-tidy] Add bugprone-suspicious-memset-usage check..
Jul 14 2017, 4:18 AM · Restricted Project

Jul 12 2017

whisperity resigned from D32700: [clang-tidy] Add bugprone-suspicious-memset-usage check..
Jul 12 2017, 11:53 PM · Restricted Project
whisperity requested changes to D32700: [clang-tidy] Add bugprone-suspicious-memset-usage check..

Considering the results published in the opening description:

Jul 12 2017, 5:05 AM · Restricted Project

Jul 11 2017

whisperity updated subscribers of D35257: [clang-tidy] Add new modernize use unary assert check.
Jul 11 2017, 8:14 AM · Restricted Project
whisperity updated subscribers of D35068: [analyzer] Detect usages of unsafe I/O functions.
Jul 11 2017, 2:25 AM

Jul 6 2017

whisperity added a comment to D34512: Add preliminary Cross Translation Unit support library.

It looks like Richard approved libTooling as a dependency for clang on the mailing list (http://lists.llvm.org/pipermail/cfe-dev/2017-July/054536.html).
If it is ok to have this code in libTooling (for now), I think we could start/continue the review of this patch.

I read that somewhat differently? It seems like Richard basically proposes adding a new library for things that control how we run clang in a multi-TU scenario. I'd call it libIndex, but that already exists :)

No, but since Richard said he did not see any justifications to include this in libTooling, I had the impression this is not his final word, and in case you see it justified, this remains the suggested direction.
But in this case I will rewrite this patch to create a new library. Are you still interested in reviewing it? Do you have any other name in mind? What about libCrossTU?

Jul 6 2017, 6:01 AM
whisperity updated the summary of D35051: [clang-tidy] Add bugprone-undefined-memory-manipulation check..
Jul 6 2017, 5:57 AM · Restricted Project
whisperity updated subscribers of D35051: [clang-tidy] Add bugprone-undefined-memory-manipulation check..
Jul 6 2017, 5:52 AM · Restricted Project
whisperity updated subscribers of D33844: [clang-tidy] terminating continue check.
Jul 6 2017, 5:41 AM · Restricted Project
whisperity updated subscribers of D33841: [clang-tidy] redundant keyword check.
Jul 6 2017, 5:40 AM · Restricted Project
whisperity updated the summary of D33826: [clang-tidy] avoid pointer cast to more strict alignment check.
Jul 6 2017, 5:29 AM · Restricted Project
whisperity updated the summary of D33829: [clang-tidy] avoid reserved names check.
Jul 6 2017, 5:27 AM · Restricted Project
whisperity updated the summary of D33825: [clang-tidy] signal handler must be plain old function check.
Jul 6 2017, 5:22 AM · Restricted Project

Jun 29 2017

whisperity updated the summary of D34508: [Analyzer] Bug Reporter Visitor to Display Values of Variables - PRELIMINARY!.
Jun 29 2017, 2:54 AM

Jun 23 2017

whisperity updated subscribers of D34512: Add preliminary Cross Translation Unit support library.
Jun 23 2017, 3:58 AM

Jun 22 2017

whisperity updated subscribers of D34512: Add preliminary Cross Translation Unit support library.
Jun 22 2017, 12:51 PM

May 31 2017

whisperity updated subscribers of D33672: [analyzer] INT50-CPP. Do not cast to an out-of-range enumeration checker.
May 31 2017, 4:05 AM

May 12 2017

whisperity added a project to D20689: [clang-tidy] Suspicious Call Argument checker: Restricted Project.
May 12 2017, 2:40 AM · Restricted Project

May 9 2017

whisperity retitled D32350: [Analyzer] Exception checker for misuse: uncaught/noncompliant throws from [Analyzer] Exception Checker to [Analyzer] Exception checker for misuse: uncaught/noncompliant throws.
May 9 2017, 2:20 AM
whisperity updated the summary of D32350: [Analyzer] Exception checker for misuse: uncaught/noncompliant throws.
May 9 2017, 2:00 AM
whisperity added inline comments to D32350: [Analyzer] Exception checker for misuse: uncaught/noncompliant throws.
May 9 2017, 1:58 AM
whisperity updated subscribers of D32350: [Analyzer] Exception checker for misuse: uncaught/noncompliant throws.
May 9 2017, 1:38 AM

May 2 2017

whisperity updated subscribers of D27918: [analyzer] OStreamChecker.
May 2 2017, 12:05 PM
whisperity updated subscribers of D32743: [clang-tidy] Add new cert-dcl21-cpp check. .
May 2 2017, 4:26 AM · Restricted Project
whisperity updated the summary of D32700: [clang-tidy] Add bugprone-suspicious-memset-usage check..
May 2 2017, 12:43 AM · Restricted Project
whisperity updated subscribers of D32700: [clang-tidy] Add bugprone-suspicious-memset-usage check..
May 2 2017, 12:30 AM · Restricted Project

Apr 28 2017

whisperity updated subscribers of D32592: [Analyzer] Iterator Checker - Part 1: Minimal Checker for a Simple Test Case.
Apr 28 2017, 6:23 AM
whisperity updated subscribers of D32642: [Analyzer] Iterator Checker - Part 2: Increment, decrement operators and ahead-of-begin checks.
Apr 28 2017, 6:23 AM

Apr 25 2017

whisperity added inline comments to D30691: [analyzer] Support for naive cross translational unit analysis.
Apr 25 2017, 6:11 AM

Apr 12 2017

whisperity updated subscribers of D31975: [Analyzer] Iterator Checkers.
Apr 12 2017, 5:09 AM

Mar 10 2017

whisperity updated subscribers of D30831: [ASTImporter] Import fix of GCCAsmStmts w/ missing symbolic operands.
Mar 10 2017, 8:28 AM

Mar 7 2017

whisperity updated subscribers of D30691: [analyzer] Support for naive cross translational unit analysis.
Mar 7 2017, 8:19 AM