diff options
-rw-r--r-- | src/Args.cpp | 2 | ||||
-rw-r--r-- | unittest/test_Args.cpp | 14 |
2 files changed, 12 insertions, 4 deletions
diff --git a/src/Args.cpp b/src/Args.cpp index 50cfa672..da041bcb 100644 --- a/src/Args.cpp +++ b/src/Args.cpp @@ -77,7 +77,7 @@ Args::from_atfile(const std::string& filename, AtFileFormat format) } break; case AtFileFormat::msvc: - if (*pos != '"' && *pos != '\\') { + if (*pos != '"') { pos--; } break; diff --git a/unittest/test_Args.cpp b/unittest/test_Args.cpp index 8ae2470c..f5d9ae7b 100644 --- a/unittest/test_Args.cpp +++ b/unittest/test_Args.cpp @@ -139,12 +139,12 @@ TEST_CASE("Args::from_atfile") CHECK(args[6] == "seve\nth"); } - SUBCASE("Only escape double quote and backslash in alternate format") + SUBCASE("Only escape double quote in alternate format") { - util::write_file("at_file", "\"\\\"\\a\\ \\\\\\ \\b\\\"\"\\"); + util::write_file("at_file", "\"\\\"\\a\\ \\b\\\"\"\\"); args = *Args::from_atfile("at_file", Args::AtFileFormat::msvc); CHECK(args.size() == 1); - CHECK(args[0] == "\"\\a\\ \\\\ \\b\"\\"); + CHECK(args[0] == "\"\\a\\ \\b\"\\"); } SUBCASE("Ignore single quote in alternate format") @@ -155,6 +155,14 @@ TEST_CASE("Args::from_atfile") CHECK(args[0] == "'a"); CHECK(args[1] == "b'"); } + + SUBCASE("Do not escape backslash in alternate format") + { + util::write_file("at_file", "\"-DDIRSEP='\\\\'\""); + args = *Args::from_atfile("at_file", Args::AtFileFormat::msvc); + CHECK(args.size() == 1); + CHECK(args[0] == "-DDIRSEP='\\\\'"); + } } TEST_CASE("Args copy assignment operator") |