Skip to content

Commit 728d776

Browse files
committed
Remove restriction to C++ 20
1 parent 25e62bd commit 728d776

File tree

3 files changed

+7
-6
lines changed

3 files changed

+7
-6
lines changed

clang-tools-extra/clang-tidy/modernize/UseDesignatedInitializersCheck.h

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -28,9 +28,6 @@ class UseDesignatedInitializersCheck : public ClangTidyCheck {
2828
std::optional<TraversalKind> getCheckTraversalKind() const override {
2929
return TK_IgnoreUnlessSpelledInSource;
3030
}
31-
bool isLanguageVersionSupported(const LangOptions &LangOpts) const override {
32-
return LangOpts.CPlusPlus20;
33-
}
3431
void storeOptions(ClangTidyOptions::OptionMap &Opts) override;
3532

3633
private:

clang-tools-extra/docs/clang-tidy/checks/modernize/use-designated-initializers.rst

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,11 @@ arise when changing the order of fields.
1212

1313
C++ 20 supports the designated initializer syntax for aggregate types.
1414
By applying it, we can always be sure that aggregates are constructed correctly,
15-
because the every variable being initialized is referenced by name.
15+
because every variable being initialized is referenced by name.
16+
17+
Even when compiling in a language version older than C++ 20, depending on you compiler,
18+
designated initializers are potentially supported. Therefore, the check is not restricted
19+
to C++ 20 and older.
1620

1721
Example:
1822

clang-tools-extra/test/clang-tidy/checkers/modernize/use-designated-initializers.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
1-
// RUN: %check_clang_tidy -std=c++20 %s modernize-use-designated-initializers %t
2-
// RUN: %check_clang_tidy -check-suffixes=,SINGLE-ELEMENT -std=c++20 %s modernize-use-designated-initializers %t \
1+
// RUN: %check_clang_tidy -std=c++17 %s modernize-use-designated-initializers %t
2+
// RUN: %check_clang_tidy -check-suffixes=,SINGLE-ELEMENT -std=c++17 %s modernize-use-designated-initializers %t \
33
// RUN: -- -config="{CheckOptions: [{key: modernize-use-designated-initializers.IgnoreSingleElementAggregates, value: false}]}" \
44
// RUN: --
55

0 commit comments

Comments
 (0)