balazske added inline comments.
================ Comment at: clang/include/clang/StaticAnalyzer/Core/PathSensitive/CheckerContext.h:194 + const ProgramPointTag *Tag = nullptr) { + // Say this 3 times fast. + State = State ? State : getState(); ---------------- martong wrote: > I like joke :D But there are people who may not, so, the code would be more > pro without it. ``` if (!State) State = getState(); ``` is better? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D77866/new/ https://reviews.llvm.org/D77866 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits