[clang-tools-extra] [clangd] Clean formatting modernize-use-override (PR #81435)

2024-02-12 Thread Kevin Joseph via cfe-commits
kevinjoseph1995 wrote: > Small fix in release notes needed. If with this change tests are passing, > then it's looking fine for me. ninja check-clang-tools reports the following: ``` Total Discovered Tests: 2813 Unsupported :7 (0.25%) Passed : 2804 (99.68%) Expectedly

[clang-tools-extra] [clangd] Clean formatting modernize-use-override (PR #81435)

2024-02-12 Thread Kevin Joseph via cfe-commits
@@ -160,6 +160,10 @@ Changes in existing checks `AllowStringArrays` option, enabling the exclusion of array types with deduced length initialized from string literals. +- Improved :doc:` kevinjoseph1995 wrote: Thanks, added the missing check name.

[clang-tools-extra] [clangd] Clean formatting modernize-use-override (PR #81435)

2024-02-12 Thread Kevin Joseph via cfe-commits
https://github.com/kevinjoseph1995 updated https://github.com/llvm/llvm-project/pull/81435 >From db2c4ee74ffb0592ec7f3fd5557dbb5399ef998d Mon Sep 17 00:00:00 2001 From: Kevin Joseph Date: Sun, 11 Feb 2024 13:39:51 -0800 Subject: [PATCH 1/3] [clangd] Clean formatting modernize-use-override

[clang-tools-extra] [clangd] Clean formatting modernize-use-override (PR #81435)

2024-02-12 Thread Kevin Joseph via cfe-commits
kevinjoseph1995 wrote: > Release notes entry (for clang-tidy) is also missing. Added a small entry to the Release notes. I wasn't sure about which section to add it to. Please let me know me if that isn't the right one. https://github.com/llvm/llvm-project/pull/81435

[clang-tools-extra] [clangd] Clean formatting modernize-use-override (PR #81435)

2024-02-11 Thread Kevin Joseph via cfe-commits
@@ -898,6 +898,34 @@ TEST(DiagnosticTest, ClangTidySelfContainedDiags) { withFix(equalToFix(ExpectedDFix)); } +TEST(DiagnosticTest, ClangTidySelfContainedDiagsFormatting) { kevinjoseph1995 wrote: Added a case in

[clang-tools-extra] [clangd] Clean formatting modernize-use-override (PR #81435)

2024-02-11 Thread Kevin Joseph via cfe-commits
https://github.com/kevinjoseph1995 updated https://github.com/llvm/llvm-project/pull/81435 >From db2c4ee74ffb0592ec7f3fd5557dbb5399ef998d Mon Sep 17 00:00:00 2001 From: Kevin Joseph Date: Sun, 11 Feb 2024 13:39:51 -0800 Subject: [PATCH 1/2] [clangd] Clean formatting modernize-use-override

[clang-tools-extra] [clangd] Clean formatting modernize-use-override (PR #81435)

2024-02-11 Thread Kevin Joseph via cfe-commits
https://github.com/kevinjoseph1995 created https://github.com/llvm/llvm-project/pull/81435 When applying the recommended fix for the "modernize-use-override" clang-tidy diagnostic there was a stray whitespace. This PR fixes that. Resolves https://github.com/clangd/clangd/issues/1704 >From