[PATCH] D110436: [WIP] Add %n format specifier warning

2021-09-29 Thread Jayson Yan via Phabricator via cfe-commits
Jaysonyan updated this revision to Diff 375887. Jaysonyan added a comment. - Add more verbose warning message Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110436/new/ https://reviews.llvm.org/D110436 Files: clang/include/clang/Basic/DiagnosticG

[PATCH] D110436: [WIP] Add %n format specifier warning

2021-09-24 Thread Roman Lebedev via Phabricator via cfe-commits
lebedev.ri added a comment. The patch description says what the change is, but not why it is the way it is. In particular, it might be helpful to be more verbose than just stating that something is unsafe. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.o

[PATCH] D110436: [WIP] Add %n format specifier warning

2021-09-24 Thread Jayson Yan via Phabricator via cfe-commits
Jaysonyan created this revision. Jaysonyan added reviewers: leonardchan, phosek. Jaysonyan requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This warning is to dissuade developers from using the potentially unsafe %n format specifier. Current