[PATCH] D56136: [compiler-rt] [sanitizer_common] Add tests for more stdio.h functions

[Kamil Rytarowski via Phabricator via cfe-commits]

krytarowski added inline comments.


================
Comment at: test/sanitizer_common/TestCases/Posix/feof_fileno_ferror.cc:7
+int main(int argc, char **argv) {
+  FILE *fp;
+  int fd;
----------------
Please move local variable declaration to the first line used (whenever possible)


================
Comment at: test/sanitizer_common/TestCases/Posix/fgetc_ungetc_getc.cc:6
+int main(int argc, char **argv) {
+  FILE *fp;
+
----------------
`FILE *fp = fp = fopen(argv[0], "r");`


Repository:
  rCRT Compiler Runtime

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

https://reviews.llvm.org/D56136