[PATCH] D73897: [analyzer] StdLibraryFunctionsChecker refactor: remove macros

Fri Feb 7 07:42:17 PST 2020

Szelethus added inline comments.

================
Comment at: clang/lib/StaticAnalyzer/Checkers/StdLibraryFunctionsChecker.cpp:42-49
 // The following standard C functions are currently supported:
 //
 //   fgetc      getline   isdigit   isupper
 //   fread      isalnum   isgraph   isxdigit
 //   fwrite     isalpha   islower   read
 //   getc       isascii   isprint   write
 //   getchar    isblank   ispunct
----------------
martong wrote:
> Szelethus wrote:
> > I would prefer to just have a checker option that could print out the currently modeled function rather than these lines of a recipe for outdated comments.
> Yes I agree, especially because I am planning to add a plethora of new functions in the future. I think that would be the appropriate time to implement the checker option.
Totally agreed! Thank you for the cleanup!

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D73897/new/

https://reviews.llvm.org/D73897