Ignore:
Timestamp:
Jul 17, 2017, 12:28:51 PM (8 years ago)
Author:
Frederik Heber <frederik.heber@…>
Branches:
Action_Thermostats, ForceAnnealing_with_BondGraph_continued, ForceAnnealing_with_BondGraph_continued_betteresults
Children:
3bd4a66
Parents:
9346af
git-author:
Frederik Heber <frederik.heber@…> (07/06/17 22:18:13)
git-committer:
Frederik Heber <frederik.heber@…> (07/17/17 12:28:51)
Message:

Query::handle() no longer returns bool but has internal result flag.

  • we use this flag conditionally in setResult(), i.e. if the handle() has failed, then we should not set its result which might overwrite a present default value in the parameter.
  • this fixes the problem with StepWorldTime which has a default value of 1 but which was overwritten with 0 because of the non-conditionally calling of setResult().
  • this required change of "output-types" default parameter to an empty vector. So far, we were just lucky that this actually worked.
  • also StoreSaturatedFragmentAction needed the same change as default values have to be consistent over the specific token.
File:
1 edited

Legend:

Unmodified
Added
Removed
  • src/UIElements/CommandLineUI/Query/KeyValuePairCommandLineQuery.cpp

    r9346af rbf87c2  
    4949CommandLineDialog::KeyValuePairCommandLineQuery::~KeyValuePairCommandLineQuery() {}
    5050
    51 bool CommandLineDialog::KeyValuePairCommandLineQuery::handle() {
     51void CommandLineDialog::KeyValuePairCommandLineQuery::handle() {
    5252  if (CommandLineParser::getInstance().vm.count(getTitle())) {
    5353    try {
     
    5555    } catch(boost::bad_any_cast &e) {
    5656      temp = KeyValuePair(std::string(""));
    57       return false;
     57      return;
    5858    }
    59     return true;
     59    handleSuccess = true; return;
    6060  }
    61   return false;
    6261}
    6362
Note: See TracChangeset for help on using the changeset viewer.