mydeveloperday wrote:
> Where is the documentation for the value? CLion reports `clang-format` option
> as an error by the reason. No doc, no schema rule.
Personally, I'm not a fan of CLion enough to care! especially following
JetBrains slating of clang-format: (thanks for doing that at a mas
HazardyKnusperkeks wrote:
> Where is the documentation for the value? CLion reports `clang-format` option
> as an error by the reason. No doc, no schema rule.
There:
> I don't want to "advertise" this new style in the documentation but am
> willing to be overruled by the majority.
There is no
alexey-utkin wrote:
Where is the documentation for the value? CLion reports `clang-format` option
as an error by the reason. No doc, no schema rule.
https://github.com/llvm/llvm-project/pull/69814
___
cfe-commits mailing list
cfe-commits@lists.llvm.o
https://github.com/owenca closed https://github.com/llvm/llvm-project/pull/69814
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/owenca updated
https://github.com/llvm/llvm-project/pull/69814
>From 78a2661ba7a18ecf8b09be71f6959fbc9fcf70fe Mon Sep 17 00:00:00 2001
From: Owen Pan
Date: Fri, 20 Oct 2023 23:01:38 -0700
Subject: [PATCH 1/2] [clang-format] Add a new style for the clang-format
source code
-
owenca wrote:
Yep. Each file will contain only `BasedOnStyle: clang-format`.
https://github.com/llvm/llvm-project/pull/69814
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
HazardyKnusperkeks wrote:
> > What is your goal with that style?
>
> It was actually your [idea](https://reviews.llvm.org/D153208#4431239). 😉
I know, but I thought the thought was dropped. :)
So the intent is to use it on our `.clang-format` files?
https://github.com/llvm/llvm-project/pull/69
owenca wrote:
I was wondering myself. Will undo it before merging.
https://github.com/llvm/llvm-project/pull/69814
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/rymiel approved this pull request.
https://github.com/llvm/llvm-project/pull/69814
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
rymiel wrote:
> I don't want to "advertise" this new style in the documentation but am
> willing to be overruled by the majority.
But you added it to the help text?
(I'm fine with this being undocumented)
https://github.com/llvm/llvm-project/pull/69814
owenca wrote:
> What is your goal with that style?
It was actually your [idea](https://reviews.llvm.org/D153208#4431239). 😉
https://github.com/llvm/llvm-project/pull/69814
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org
https://github.com/HazardyKnusperkeks approved this pull request.
What is your goal with that style?
https://github.com/llvm/llvm-project/pull/69814
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo
owenca wrote:
I don't want to "advertise" this new style in the documentation but am willing
to be overruled by the majority.
https://github.com/llvm/llvm-project/pull/69814
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.or
https://github.com/owenca created
https://github.com/llvm/llvm-project/pull/69814
None
>From 78a2661ba7a18ecf8b09be71f6959fbc9fcf70fe Mon Sep 17 00:00:00 2001
From: Owen Pan
Date: Fri, 20 Oct 2023 23:01:38 -0700
Subject: [PATCH] [clang-format] Add a new style for the clang-format source
code
14 matches
Mail list logo
