feedc0de/qt-creator
1
0
Fork 0
forked from qt-creator/qt-creator
Code Pull Requests Activity

CppEditor: Add missing hyphen before option names

Browse Source 
Amends 06e2f8d8fa.

Change-Id: I514f04ce21b91c80dc15d1e0323b2d80df08aeb5
Reviewed-by: <github-actions-qt-creator@cristianadam.eu>
Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
Reviewed-by: Hannes Domani <ssbssa@yahoo.de>
Reviewed-by: David Schulz <david.schulz@qt.io>
This commit is contained in:
Christian Kandeler
2022-08-08 15:37:45 +02:00
parent 176c3906fe
commit ff2235bd5d
2 changed files with 7 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/plugins/cppeditor/compileroptionsbuilder.cpp
View File

@@ -473,16 +473,16 @@ void CompilerOptionsBuilder::addLanguageVersionAndExtensions()
default:
break;
case LanguageVersion::CXX14:
option = "-clang:std=c++14";
option = "-clang:-std=c++14";
break;
case LanguageVersion::CXX17:
option = "-clang:std=c++17";
option = "-clang:-std=c++17";
break;
case LanguageVersion::CXX20:
option = "-clang:std=c++20";
option = "-clang:-std=c++20";
break;
case LanguageVersion::CXX2b:
option = "-clang:std=c++2b";
option = "-clang:-std=c++2b";
break;
}

6
src/plugins/cppeditor/compileroptionsbuilder_test.cpp
View File

@@ -192,7 +192,7 @@ void CompilerOptionsBuilderTest::testLanguageVersionIsExplicitlySetIfNotProvided
UseTweakedHeaderPaths::No, UseLanguageDefines::Yes};
compilerOptionsBuilder.build(ProjectFile::CXXSource, UsePrecompiledHeaders::No);
QVERIFY(compilerOptionsBuilder.options().contains("-clang:std=c++17"));
QVERIFY(compilerOptionsBuilder.options().contains("-clang:-std=c++17"));
}
void CompilerOptionsBuilderTest::testAddWordWidth()
@@ -633,7 +633,7 @@ void CompilerOptionsBuilderTest::testBuildAllOptionsMsvc()
[&t](const QString &o) { return o.contains(t.toNative("wrappedQtHeaders/QtCore")); });
QCOMPARE(compilerOptionsBuilder.options(),
(QStringList{"-nostdinc", "-nostdinc++", "--driver-mode=cl", "/Zs", "-m64",
"--target=x86_64-apple-darwin10", "/TP", "-clang:std=c++17",
"--target=x86_64-apple-darwin10", "/TP", "-clang:-std=c++17",
"-fms-compatibility-version=19.00", "-DprojectFoo=projectBar",
"-D__FUNCSIG__=\"void __cdecl someLegalAndLongishFunctionNameThatWorksAroundQTCREATORBUG-24580(void)\"",
"-D__FUNCTION__=\"someLegalAndLongishFunctionNameThatWorksAroundQTCREATORBUG-24580\"",
@@ -662,7 +662,7 @@ void CompilerOptionsBuilderTest::testBuildAllOptionsMsvcWithExceptions()
[&t](const QString &o) { return o.contains(t.toNative("wrappedQtHeaders/QtCore")); });
QCOMPARE(compilerOptionsBuilder.options(),
(QStringList{"-nostdinc", "-nostdinc++", "--driver-mode=cl", "/Zs", "-m64",
"--target=x86_64-apple-darwin10", "/TP", "-clang:std=c++17", "-fcxx-exceptions",
"--target=x86_64-apple-darwin10", "/TP", "-clang:-std=c++17", "-fcxx-exceptions",
"-fexceptions", "-fms-compatibility-version=19.00",
"-DprojectFoo=projectBar",
"-D__FUNCSIG__=\"void __cdecl someLegalAndLongishFunctionNameThatWorksAroundQTCREATORBUG-24580(void)\"",