[PATCH] D158285: [NFC][CLANG] Fix wrong orders of function arguments positions
Tom Honermann via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Aug 21 07:55:44 PDT 2023
tahonermann added a comment.
> To me, an important metric of a quality patch is if the title and the summary describe why we have this patch, and what we do about it.
@steakhal, I'll work with the people submitting these patches to add more context to the title, summary, and commit comments. I agree that would be helpful.
Ideally, we would copy the report from the static analysis tool, but we have been informed that we are not allowed to divulge the tool we are using, so we aren't permitted to do that.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D158285/new/
https://reviews.llvm.org/D158285
More information about the cfe-commits
mailing list