[PATCH] D101387: remove single quotes around sugguestion diagnostic
Richard Smith - zygoloid via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Apr 27 23:42:48 PDT 2021
rsmith added a comment.
Note that, per https://clang.llvm.org/docs/InternalsManual.html#the-format-string,
> Diagnostics should never take random English strings as arguments: you shouldn’t use “you have a problem with %0” and pass in things like “your argument” or “your return value” as arguments.
... so this diagnostic needs to be fundamentally restructured: either it should be split into multiple diagnostics for the various individual messages, or a `%select` should be added with the appropriate message text, or similar.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D101387/new/
https://reviews.llvm.org/D101387
More information about the cfe-commits
mailing list