[clang] [clang-format] Use double hyphen for multiple-letter flags (PR #100978)

2024-08-21 Thread via cfe-commits
github-actions[bot] wrote: @magic-akari Congratulations on having your first Pull Request (PR) merged into the LLVM Project! Your changes will be combined with recent changes from other authors, then tested by our [build bots](https://lab.llvm.org/buildbot/). If there is a problem with a bu

[clang] [clang-format] Use double hyphen for multiple-letter flags (PR #100978)

2024-08-21 Thread Björn Schäpers via cfe-commits
https://github.com/HazardyKnusperkeks closed https://github.com/llvm/llvm-project/pull/100978 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-format] Use double hyphen for multiple-letter flags (PR #100978)

2024-08-01 Thread via cfe-commits
https://github.com/mydeveloperday approved this pull request. https://github.com/llvm/llvm-project/pull/100978 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-format] Use double hyphen for multiple-letter flags (PR #100978)

2024-07-29 Thread Björn Schäpers via cfe-commits
HazardyKnusperkeks wrote: Okay, I can't revoke my vote and remove me as reviewer. As demonstrated I'm not qualified to review python changes. https://github.com/llvm/llvm-project/pull/100978 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http

[clang] [clang-format] Use double hyphen for multiple-letter flags (PR #100978)

2024-07-29 Thread Björn Schäpers via cfe-commits
HazardyKnusperkeks wrote: My bad, I thought you changed the options to those scripts. https://github.com/llvm/llvm-project/pull/100978 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-format] Use double hyphen for multiple-letter flags (PR #100978)

2024-07-29 Thread via cfe-commits
magic-akari wrote: > You certainly can add the double hyphen flags, but you can't remove the > single ones. There seems to be a misunderstanding. Please review the original issue. I did _not_ eliminate support for single hyphens in clang-format. What I did was modify the scripts that call c

[clang] [clang-format] Use double hyphen for multiple-letter flags (PR #100978)

2024-07-29 Thread Björn Schäpers via cfe-commits
https://github.com/HazardyKnusperkeks requested changes to this pull request. You certainly can add the double hyphen flags, but you can't remove the single ones. https://github.com/llvm/llvm-project/pull/100978 ___ cfe-commits mailing list cfe-commit

[clang] [clang-format] Use double hyphen for multiple-letter flags (PR #100978)

2024-07-29 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-format Author: None (magic-akari) Changes - Closes: #100974 --- Full diff: https://github.com/llvm/llvm-project/pull/100978.diff 5 Files Affected: - (modified) clang/tools/clang-format/clang-format-diff.py (+4-4) - (modified) clang/tools/clan

[clang] [clang-format] Use double hyphen for multiple-letter flags (PR #100978)

2024-07-29 Thread via cfe-commits
https://github.com/magic-akari ready_for_review https://github.com/llvm/llvm-project/pull/100978 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-format] Use double hyphen for multiple-letter flags (PR #100978)

2024-07-28 Thread via cfe-commits
github-actions[bot] wrote: Thank you for submitting a Pull Request (PR) to the LLVM Project! This PR will be automatically labeled and the relevant teams will be notified. If you wish to, you can add reviewers by using the "Reviewers" section on this page. If this is not working for you, it

[clang] [clang-format] Use double hyphen for multiple-letter flags (PR #100978)

2024-07-28 Thread via cfe-commits
https://github.com/magic-akari created https://github.com/llvm/llvm-project/pull/100978 - Closes: #100974 >From 4c90d85b5edf2ca93fe28e503f3eaa3d6dad4cb3 Mon Sep 17 00:00:00 2001 From: magic-akari Date: Mon, 29 Jul 2024 14:30:40 +0800 Subject: [PATCH] [clang-format] Use double hyphen for multi