diff options
| author | David Cole <david.cole@kitware.com> | 2012-08-20 19:39:35 (GMT) |
|---|---|---|
| committer | CMake Topic Stage <kwrobot@kitware.com> | 2012-08-20 19:39:35 (GMT) |
| commit | 9f555e805dd025f6428e5dff7f4a928c5dc5fbbb (patch) | |
| tree | 97c3714aec7c6e68e8f792d32c3703b0d0a727a4 /Source | |
| parent | 14c8096ba88502409f076cf0b0e5fe471c18f11b (diff) | |
| parent | 08c71acc8729aa51b389c0583bca351077e5e44e (diff) | |
| download | CMake-9f555e805dd025f6428e5dff7f4a928c5dc5fbbb.zip CMake-9f555e805dd025f6428e5dff7f4a928c5dc5fbbb.tar.gz CMake-9f555e805dd025f6428e5dff7f4a928c5dc5fbbb.tar.bz2 | |
Merge topic 'list-command-argument-fixes'
08c71ac CMake.List test: explicitely test with lists containing only an empty string
52d9c38 list command: error on too many arguments
Diffstat (limited to 'Source')
| -rw-r--r-- | Source/cmListCommand.cxx | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/Source/cmListCommand.cxx b/Source/cmListCommand.cxx index 9d4f175..7848424 100644 --- a/Source/cmListCommand.cxx +++ b/Source/cmListCommand.cxx @@ -429,6 +429,12 @@ bool cmListCommand this->SetError("sub-command REVERSE requires a list as an argument."); return false; } + else if(args.size() > 2) + { + this->SetError( + "sub-command REVERSE only takes one argument."); + return false; + } const std::string& listName = args[1]; // expand the variable @@ -463,6 +469,12 @@ bool cmListCommand "sub-command REMOVE_DUPLICATES requires a list as an argument."); return false; } + else if(args.size() > 2) + { + this->SetError( + "sub-command REMOVE_DUPLICATES only takes one argument."); + return false; + } const std::string& listName = args[1]; // expand the variable @@ -506,6 +518,12 @@ bool cmListCommand this->SetError("sub-command SORT requires a list as an argument."); return false; } + else if(args.size() > 2) + { + this->SetError( + "sub-command SORT only takes one argument."); + return false; + } const std::string& listName = args[1]; // expand the variable |