From aeddf63587c242a7995598f4b1a5f248e4e3cb13 Mon Sep 17 00:00:00 2001 From: Brad King Date: Thu, 11 Apr 2019 10:41:03 -0400 Subject: cmArgumentParser: Fix -Wcomma warning Clang `-Wcomma` warns: ``` Source/cmArgumentParser.cxx:58:42: warning: possible misuse of comma operator this->CurrentList = (val.emplace_back(), &val.back()); ^ ``` This was introduced by commit 4359fe133b (Introduce cmArgumentParser, 2019-03-23). Suppress it with the suggested cast. --- Source/cmArgumentParser.cxx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Source/cmArgumentParser.cxx b/Source/cmArgumentParser.cxx index 9a9932c..751d117 100644 --- a/Source/cmArgumentParser.cxx +++ b/Source/cmArgumentParser.cxx @@ -55,7 +55,7 @@ void Instance::Bind(StringList& val) void Instance::Bind(MultiStringList& val) { this->CurrentString = nullptr; - this->CurrentList = (val.emplace_back(), &val.back()); + this->CurrentList = (static_cast(val.emplace_back()), &val.back()); this->ExpectValue = false; } -- cgit v0.12