Eugene.Zelenko added inline comments.
================
Comment at:
clang-tools-extra/clang-tidy/bugprone/SprintfWithFixedSizeBufferCheck.h:29
+ void check(const ast_matchers::MatchFinder::MatchResult &Result) override;
+};
+
----------------
Yoorkin wrote:
> Eugene.Zelenko wrote:
> > Please add `isLanguageVersionSupported` (C and C++).
> can i just simply use
> ```
> bool isLanguageVersionSupported(const LangOptions &LangOpts) const override
> {
> return LangOpts.C99 || LangOpts.CPlusPlus11;
> }
> ```
> I think the check should be supported after C99 and Cpp11 standards. But i
> don't know whether it return true when language version is c++20 or C11
`snprintf` is part of standard C library, so it could be used in early C and
C++ standards.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D132294/new/
https://reviews.llvm.org/D132294
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
