mirror/CMake
1
0
Fork 0
mirror of https://github.com/Kitware/CMake.git synced 2026-01-26 08:29:54 -06:00
Code Issues Packages Projects Releases Wiki Activity

cmArgumentParser: Fix -Wcomma warning

Browse Source 
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.
This commit is contained in:
Brad King
2019-04-11 10:41:03 -04:00
parent a550e2d6e4
commit aeddf63587
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
Source/cmArgumentParser.cxx
View File

@@ -55,7 +55,7 @@ void Instance::Bind(StringList& val)
void Instance::Bind(MultiStringList& val) void Instance::Bind(MultiStringList& val)
{ {
this->CurrentString = nullptr; this->CurrentString = nullptr;
this->CurrentList = (val.emplace_back(), &val.back()); this->CurrentList = (static_cast<void>(val.emplace_back()), &val.back());
this->ExpectValue = false; this->ExpectValue = false;
} }