Szelethus accepted this revision.
Szelethus added a comment.
This revision is now accepted and ready to land.
I'd prefer if you moved `f_leak_2` to `stream-notes.c`. Otherwise, LGTM.
================
Comment at: clang/test/Analysis/stream.c:147-150
+ FILE *p1 = fopen("foo1.c", "r");
+ if (!p1)
+ return;
+ FILE *p2 = fopen("foo2.c", "r");
----------------
I'd prefer to see notes from D81407 here.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82845/new/
https://reviews.llvm.org/D82845
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
