thorsten-klein wrote:
@EugeneZelenko What do you think about some additional option
`EnforceNoNamespaceComments` in order to enforce that no namespace comment is
present?
If you like the idea, I could create some additional PR for introducing this
option.
https://github.com/llvm/llvm-project/
https://github.com/thorsten-klein edited
https://github.com/llvm/llvm-project/pull/124265
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/thorsten-klein edited
https://github.com/llvm/llvm-project/pull/124265
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/thorsten-klein updated
https://github.com/llvm/llvm-project/pull/124265
>From 9f3bc5021e6da87fa9a8db60b513acb87ed6b02f Mon Sep 17 00:00:00 2001
From: "Klein, Thorsten (GDE-EDSI1)"
Date: Fri, 24 Jan 2025 13:46:24 +0100
Subject: [PATCH] added option AllowNoNamespaceComments for
@@ -287,6 +287,11 @@ Changes in existing checks
` check to
report a location even when the member location is not valid.
+- Improved :doc:`google-readability-namespace-comments
+ ` check to
+ add an option ``AllowNoNamespaceComments``, that permits
thors
https://github.com/thorsten-klein updated
https://github.com/llvm/llvm-project/pull/124265
>From bade51bc7f0135e5c74cc64ee851a17813c6820f Mon Sep 17 00:00:00 2001
From: "Klein, Thorsten (GDE-EDSI1)"
Date: Fri, 24 Jan 2025 13:46:24 +0100
Subject: [PATCH] added option AllowNoNamespaceComments for
https://github.com/thorsten-klein updated
https://github.com/llvm/llvm-project/pull/124265
>From 15a8d29ef422afe853adeb1fd66e5218d9ebfa34 Mon Sep 17 00:00:00 2001
From: "Klein, Thorsten (GDE-EDSI1)"
Date: Fri, 24 Jan 2025 13:46:24 +0100
Subject: [PATCH] added option AllowNoNamespaceComments for
https://github.com/thorsten-klein updated
https://github.com/llvm/llvm-project/pull/124265
>From 608ceb0bb475aad65abb55a1dd02bf84d62ec99c Mon Sep 17 00:00:00 2001
From: "Klein, Thorsten (GDE-EDSI1)"
Date: Fri, 24 Jan 2025 13:46:24 +0100
Subject: [PATCH] added option AllowNoNamespaceComments for
thorsten-klein wrote:
Thanks a lot for your review @vbvictor!
I have resolved your review findings.
What do you think about the option's name? Is `AllowNoNamespaceComments` fine
for you or shall I rename to `AllowOmittingNamespaceComments` (or any other
suggestion)?
https://github.com/llvm/
https://github.com/thorsten-klein created
https://github.com/llvm/llvm-project/pull/124265
new option AllowNoNamespaceComments for
`google-readability-namespace-comments.AllowNoNamespaceComments` is added.
Ref: #124264
When true, the check will allow that no namespace comment is present. If a
10 matches
Mail list logo