From 20713ab89de358540a19cb07b13202d4298fe393 Mon Sep 17 00:00:00 2001 From: Zach Mullen Date: Thu, 3 Sep 2009 11:14:13 -0400 Subject: [PATCH] Fixed warnings --- Source/CTest/cmCTestMultiProcessHandler.cxx | 2 +- Source/CTest/cmCTestTestCommand.h | 2 +- Source/CTest/cmCTestTestHandler.cxx | 5 +++++ Source/CTest/cmCTestTestHandler.h | 1 + Source/cmSetTestsPropertiesCommand.h | 7 +++++-- 5 files changed, 13 insertions(+), 4 deletions(-) diff --git a/Source/CTest/cmCTestMultiProcessHandler.cxx b/Source/CTest/cmCTestMultiProcessHandler.cxx index 0f7cd3c37..f0d926cdd 100644 --- a/Source/CTest/cmCTestMultiProcessHandler.cxx +++ b/Source/CTest/cmCTestMultiProcessHandler.cxx @@ -161,7 +161,7 @@ void cmCTestMultiProcessHandler::StartNextTests() for(TestMap::iterator i = tests.begin(); i != tests.end(); ++i) { - int processors = this->Properties[i->first]->Processors; + //int processors = this->Properties[i->first]->Processors; // if(processors > ) // start test should start only one test diff --git a/Source/CTest/cmCTestTestCommand.h b/Source/CTest/cmCTestTestCommand.h index f294e026c..9f3a2f40d 100644 --- a/Source/CTest/cmCTestTestCommand.h +++ b/Source/CTest/cmCTestTestCommand.h @@ -66,7 +66,7 @@ public: " [INCLUDE include regex] [RETURN_VALUE res] \n" " [EXCLUDE_LABEL exclude regex] \n" " [INCLUDE_LABEL label regex] \n" - " [PARALLEL_LEVEL level] )\n" + " [PARALLEL_LEVEL level]) \n" "Tests the given build directory and stores results in Test.xml. The " "second argument is a variable that will hold value. Optionally, " "you can specify the starting test number START, the ending test number " diff --git a/Source/CTest/cmCTestTestHandler.cxx b/Source/CTest/cmCTestTestHandler.cxx index 94008d12f..34b4192ed 100644 --- a/Source/CTest/cmCTestTestHandler.cxx +++ b/Source/CTest/cmCTestTestHandler.cxx @@ -1969,6 +1969,10 @@ bool cmCTestTestHandler::SetTestsProperties( { rtit->Timeout = atof(val.c_str()); } + if ( key == "EXPENSIVE" ) + { + rtit->Expensive = cmSystemTools::IsOn(val.c_str()); + } if ( key == "FAIL_REGULAR_EXPRESSION" ) { std::vector lval; @@ -2116,6 +2120,7 @@ bool cmCTestTestHandler::AddTest(const std::vector& args) test.IsInBasedOnREOptions = true; test.WillFail = false; + test.Expensive = false; test.Timeout = 0; test.Processors = 1; if (this->UseIncludeRegExpFlag && diff --git a/Source/CTest/cmCTestTestHandler.h b/Source/CTest/cmCTestTestHandler.h index 899a63863..e4b5e0736 100644 --- a/Source/CTest/cmCTestTestHandler.h +++ b/Source/CTest/cmCTestTestHandler.h @@ -96,6 +96,7 @@ public: std::map Measurements; bool IsInBasedOnREOptions; bool WillFail; + bool Expensive; double Timeout; int Index; //Requested number of process slots diff --git a/Source/cmSetTestsPropertiesCommand.h b/Source/cmSetTestsPropertiesCommand.h index 1644abe1f..f93d9dce8 100644 --- a/Source/cmSetTestsPropertiesCommand.h +++ b/Source/cmSetTestsPropertiesCommand.h @@ -68,10 +68,13 @@ public: "specified regular expressions, the test will fail.\n" " Example: PASS_REGULAR_EXPRESSION \"[^a-z]Error;ERROR;Failed\"\n" "Both PASS_REGULAR_EXPRESSION and FAIL_REGULAR_EXPRESSION expect a " - "list of regular expressions.\n"; + "list of regular expressions.\n" "PROCESSORS: Denotes the number of processors that this test will " "require. This is typically used for MPI tests, and should be used in " - "conjunction with the ctest_test PARALLEL_LEVEL option.\n"; + "conjunction with the ctest_test PARALLEL_LEVEL option.\n" + "EXPENSIVE: If set to true, this test will be run before tests that " + "are not marked as expensive. This should be used in conjunction with " + "the ctest_test PARALLEL_LEVEL option.\n"; } cmTypeMacro(cmSetTestsPropertiesCommand, cmCommand);