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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to