[clang-tools-extra] Fix #68492: point to the correct const location (PR #69103)

2023-10-29 Thread via cfe-commits
https://github.com/Da-Viper updated https://github.com/llvm/llvm-project/pull/69103 >From 354a8e4034afd82e6ea854848a86b9011e26269b Mon Sep 17 00:00:00 2001 From: Ezike Ebuka Date: Fri, 13 Oct 2023 19:27:15 +0100 Subject: [PATCH 1/5] Fix #68492: point to the correct const location --- .../read

[clang-tools-extra] Fix #68492: point to the correct const location (PR #69103)

2023-10-22 Thread via cfe-commits
https://github.com/Da-Viper updated https://github.com/llvm/llvm-project/pull/69103 >From 354a8e4034afd82e6ea854848a86b9011e26269b Mon Sep 17 00:00:00 2001 From: Ezike Ebuka Date: Fri, 13 Oct 2023 19:27:15 +0100 Subject: [PATCH 1/4] Fix #68492: point to the correct const location --- .../read

[clang-tools-extra] Fix #68492: point to the correct const location (PR #69103)

2023-10-22 Thread via cfe-commits
https://github.com/Da-Viper updated https://github.com/llvm/llvm-project/pull/69103 >From 354a8e4034afd82e6ea854848a86b9011e26269b Mon Sep 17 00:00:00 2001 From: Ezike Ebuka Date: Fri, 13 Oct 2023 19:27:15 +0100 Subject: [PATCH 1/4] Fix #68492: point to the correct const location --- .../read

[clang-tools-extra] Fix #68492: point to the correct const location (PR #69103)

2023-10-20 Thread Piotr Zegar via cfe-commits
https://github.com/PiotrZSL approved this pull request. https://github.com/llvm/llvm-project/pull/69103 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] Fix #68492: point to the correct const location (PR #69103)

2023-10-20 Thread Piotr Zegar via cfe-commits
@@ -297,6 +297,10 @@ Changes in existing checks ` check to ignore false-positive for ``if constexpr`` in lambda expression. +- Improved :doc:`readability-const-params-in-decls + ` place the hint check under PiotrZSL wrote: Or it can be just Improved xyz

[clang-tools-extra] Fix #68492: point to the correct const location (PR #69103)

2023-10-20 Thread Piotr Zegar via cfe-commits
@@ -297,6 +297,10 @@ Changes in existing checks ` check to ignore false-positive for ``if constexpr`` in lambda expression. +- Improved :doc:`readability-const-params-in-decls + ` place the hint check under PiotrZSL wrote: `Improved xyz check to place .

[clang-tools-extra] Fix #68492: point to the correct const location (PR #69103)

2023-10-16 Thread via cfe-commits
https://github.com/Da-Viper updated https://github.com/llvm/llvm-project/pull/69103 >From 354a8e4034afd82e6ea854848a86b9011e26269b Mon Sep 17 00:00:00 2001 From: Ezike Ebuka Date: Fri, 13 Oct 2023 19:27:15 +0100 Subject: [PATCH 1/3] Fix #68492: point to the correct const location --- .../read

[clang-tools-extra] Fix #68492: point to the correct const location (PR #69103)

2023-10-16 Thread via cfe-commits
@@ -293,6 +293,10 @@ Changes in existing checks ` check to enforce a stricter match with the swap function signature, eliminating false-positives. +- Improved :doc:`readability-const-params-in-decls EugeneZelenko wrote: Please keep alphabetical order (by

[clang-tools-extra] Fix #68492: point to the correct const location (PR #69103)

2023-10-15 Thread Piotr Zegar via cfe-commits
https://github.com/PiotrZSL approved this pull request. https://github.com/llvm/llvm-project/pull/69103 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] Fix #68492: point to the correct const location (PR #69103)

2023-10-15 Thread via cfe-commits
https://github.com/Da-Viper updated https://github.com/llvm/llvm-project/pull/69103 >From 354a8e4034afd82e6ea854848a86b9011e26269b Mon Sep 17 00:00:00 2001 From: Ezike Ebuka Date: Fri, 13 Oct 2023 19:27:15 +0100 Subject: [PATCH 1/2] Fix #68492: point to the correct const location --- .../read

[clang-tools-extra] Fix #68492: point to the correct const location (PR #69103)

2023-10-15 Thread via cfe-commits
https://github.com/Da-Viper edited https://github.com/llvm/llvm-project/pull/69103 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] Fix #68492: point to the correct const location (PR #69103)

2023-10-15 Thread Piotr Zegar via cfe-commits
PiotrZSL wrote: Missing release notes for this check, follow current style. Put also some description of change into merge request description. https://github.com/llvm/llvm-project/pull/69103 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http

[clang-tools-extra] Fix #68492: point to the correct const location (PR #69103)

2023-10-15 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-tidy Author: None (Da-Viper) Changes Fixes #68492 --- Full diff: https://github.com/llvm/llvm-project/pull/69103.diff 2 Files Affected: - (modified) clang-tools-extra/clang-tidy/readability/AvoidConstParamsInDecls.cpp (+27-16) - (modified) c

[clang-tools-extra] Fix #68492: point to the correct const location (PR #69103)

2023-10-15 Thread via cfe-commits
https://github.com/Da-Viper created https://github.com/llvm/llvm-project/pull/69103 Fixes #68492 >From 354a8e4034afd82e6ea854848a86b9011e26269b Mon Sep 17 00:00:00 2001 From: Ezike Ebuka Date: Fri, 13 Oct 2023 19:27:15 +0100 Subject: [PATCH] Fix #68492: point to the correct const location ---