[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