https://github.com/vbvictor created https://github.com/llvm/llvm-project/pull/170971
Before the change, when given such incorrect ordering: ``` :doc:`modernize-use-designated-initializers <modernize/use-designated-initializers>`, "Yes" :doc:`modernize-use-equals-default <modernize/use-equals-default>`, "Yes" :doc:`modernize-use-emplace <modernize/use-emplace>`, "Yes" :doc:`modernize-use-equals-delete <modernize/use-equals-delete>`, "Yes" :doc:`modernize-use-integer-sign-comparison <modernize/use-integer-sign-comparison>`, "Yes" ``` Script output suggested removing `modernize-use-equals-default` line that came before `modernize-use-emplace` but in my actual file, we can see that current ordering is different (same as 2nd part of the diff): ``` # | :doc:`modernize-use-constraints <modernize/use-constraints>`, "Yes" # | :doc:`modernize-use-default-member-init <modernize/use-default-member-init>`, "Yes" # | :doc:`modernize-use-designated-initializers <modernize/use-designated-initializers>`, "Yes" # | :doc:`modernize-use-emplace <modernize/use-emplace>`, "Yes" # | - :doc:`modernize-use-equals-default <modernize/use-equals-default>`, "Yes" # | :doc:`modernize-use-equals-delete <modernize/use-equals-delete>`, "Yes" # | :doc:`modernize-use-integer-sign-comparison <modernize/use-integer-sign-comparison>`, "Yes" # | :doc:`modernize-use-nodiscard <modernize/use-nodiscard>`, "Yes" # | --- 316,323 ---- # | :doc:`modernize-use-constraints <modernize/use-constraints>`, "Yes" # | :doc:`modernize-use-default-member-init <modernize/use-default-member-init>`, "Yes" # | :doc:`modernize-use-designated-initializers <modernize/use-designated-initializers>`, "Yes" # | + :doc:`modernize-use-equals-default <modernize/use-equals-default>`, "Yes" # | :doc:`modernize-use-emplace <modernize/use-emplace>`, "Yes" # | :doc:`modernize-use-equals-delete <modernize/use-equals-delete>`, "Yes" ``` We need to invert the diff, so the user can correctly follow its recommendations. >From 6c12051a6f16f1df3d73f1490b82b7907c725f8b Mon Sep 17 00:00:00 2001 From: Victor Baranov <[email protected]> Date: Sat, 6 Dec 2025 11:51:36 +0300 Subject: [PATCH] [clang-tidy] Show better diff when check_alphabetical_order.py fails --- .../test/clang-tidy/infrastructure/alphabetical-order.test | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/clang-tools-extra/test/clang-tidy/infrastructure/alphabetical-order.test b/clang-tools-extra/test/clang-tidy/infrastructure/alphabetical-order.test index 1a75c12f909fd..6cb0da7322f4f 100644 --- a/clang-tools-extra/test/clang-tidy/infrastructure/alphabetical-order.test +++ b/clang-tools-extra/test/clang-tidy/infrastructure/alphabetical-order.test @@ -1,5 +1,5 @@ // RUN: %python %S/../../../clang-tidy/tool/check_alphabetical_order.py -o %t.clang-tidy-checks-list.rst -// RUN: diff --strip-trailing-cr %t.clang-tidy-checks-list.rst %S/../../../docs/clang-tidy/checks/list.rst +// RUN: diff --strip-trailing-cr %S/../../../docs/clang-tidy/checks/list.rst %t.clang-tidy-checks-list.rst // RUN: %python %S/../../../clang-tidy/tool/check_alphabetical_order.py -o %t.ReleaseNotes.rst -// RUN: diff --strip-trailing-cr %t.ReleaseNotes.rst %S/../../../docs/ReleaseNotes.rst +// RUN: diff --strip-trailing-cr %S/../../../docs/ReleaseNotes.rst %t.ReleaseNotes.rst _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
