mgorny added inline comments.
================ Comment at: test/sanitizer_common/TestCases/Posix/feof_fileno_ferror.cc:38 + + fclose(fp); + return 0; ---------------- krytarowski wrote: > krytarowski wrote: > > `assert(!fclose(fp));` > alternatively `!= EOF` It will fail most likely, due to us closing the fd before. However, I'm not sure if we want to strictly rely on this error condition. 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
