diff options
| author | Craig Scott <craig.scott@crascit.com> | 2022-12-03 11:25:57 (GMT) |
|---|---|---|
| committer | Craig Scott <craig.scott@crascit.com> | 2022-12-03 11:25:57 (GMT) |
| commit | 466f9b886da1826243e222e946a48ba1428acf63 (patch) | |
| tree | bed7dc8cf83e427f95fbdc3ac84ac585f372c2fc /Source/cmTargetPropCommandBase.cxx | |
| parent | 7ea665b74da322e96dd3e7f90075143c2957728c (diff) | |
| download | CMake-466f9b886da1826243e222e946a48ba1428acf63.zip CMake-466f9b886da1826243e222e946a48ba1428acf63.tar.gz CMake-466f9b886da1826243e222e946a48ba1428acf63.tar.bz2 | |
cmTargetPropCommandBase::HandleArguments: flags must be OR'able
The flags argument is intended to support multiple flags from the
ArgumentFlags enum. Therefore, flags cannot be of enum type, it
must be an integral type that allows flags to be OR'ed together.
Update the one call site that was erroneously OR'ing multiple values
but then casting that to an ArgumentFlags, which could result in
passing a technically invalid value.
Amends: dd3482f6752d (cmTargetPropCommandBase: Restore
ArgumentFlags enum value bool logic, 2022-11-27)
Diffstat (limited to 'Source/cmTargetPropCommandBase.cxx')
| -rw-r--r-- | Source/cmTargetPropCommandBase.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Source/cmTargetPropCommandBase.cxx b/Source/cmTargetPropCommandBase.cxx index 391b954..8d2ff71 100644 --- a/Source/cmTargetPropCommandBase.cxx +++ b/Source/cmTargetPropCommandBase.cxx @@ -23,7 +23,7 @@ void cmTargetPropCommandBase::SetError(std::string const& e) bool cmTargetPropCommandBase::HandleArguments( std::vector<std::string> const& args, const std::string& prop, - ArgumentFlags flags) + unsigned int flags) { if (args.size() < 2) { this->SetError("called with incorrect number of arguments"); |