JonasToth added a comment. I think it would be more user friendly if the configured list can be a list and the `|` concatenation is done within your code.
================ Comment at: clang-tidy/bugprone/UnusedReturnValueCheck.cpp:29 + llvm::raw_svector_ostream OS(InlinedName); + auto Policy = Node.getASTContext().getPrintingPolicy(); + Policy.SuppressUnwrittenScope = true; ---------------- Don't use auto here, because the type is not written somewhere ================ Comment at: clang-tidy/bugprone/UnusedReturnValueCheck.cpp:53 + // Detect unused return values by finding CallExprs with CompoundStmt parent, + // ignoring any implicit nodes and parentheses in between + Finder->addMatcher( ---------------- Missing full stop. https://reviews.llvm.org/D41655 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
