Changes in src/Actions/CmdAction/FastParsingAction.cpp [7e37a3:bf3817]

File:

• src/Actions/CmdAction/FastParsingAction.cpp (modified) (4 diffs)

src/Actions/CmdAction/FastParsingAction.cpp ¶

@@ -5 +5 @@
  *      Author: heber
  */
+
+// include config.h
+#ifdef HAVE_CONFIG_H
+#include <config.h>
+#endif
 
 #include "Helpers/MemDebug.hpp"
@@ -28 +33 @@
 class CommandLineFastParsingState : public ActionState {
 public:
-  CommandLineFastParsingState(const bool _oldvalue, const bool _newvalue) :
-    oldvalue(_oldvalue),
-    newvalue(_newvalue)
+  CommandLineFastParsingState(bool _bool) :
+    boolean(_bool)
   {}
-  bool oldvalue;
-  bool newvalue;
+  bool boolean;
 };
 
@@ -60 +63 @@
 
 Action::state_ptr CommandLineFastParsingAction::performCall() {
+
   config *configuration = World::getInstance().getConfig();
-  bool oldvalue = configuration->FastParsing;
-  bool newvalue;
-  ValueStorage::getInstance().queryCurrentValue(NAME, newvalue);
-  configuration->FastParsing = newvalue;
+  ValueStorage::getInstance().queryCurrentValue(NAME, configuration->FastParsing);
   if (configuration->FastParsing)
     DoLog(0) && (Log() << Verbose(0) << "I won't parse trajectories." << endl);
   else
     DoLog(0) && (Log() << Verbose(0) << "I will parse trajectories." << endl);
-  return Action::state_ptr(new CommandLineFastParsingState(oldvalue, newvalue));
+  return Action::success;
 }
 
@@ -76 +77 @@
 
   config *configuration = World::getInstance().getConfig();
-  configuration->FastParsing = state->oldvalue;
-  if (configuration->FastParsing)
-    DoLog(0) && (Log() << Verbose(0) << "I won't parse trajectories." << endl);
-  else
-    DoLog(0) && (Log() << Verbose(0) << "I will parse trajectories." << endl);
+  configuration->FastParsing = state->boolean;
 
-  return Action::state_ptr(_state);
+  return Action::state_ptr(new CommandLineFastParsingState(!state->boolean));
 }
 
 Action::state_ptr CommandLineFastParsingAction::performRedo(Action::state_ptr _state){
-  CommandLineFastParsingState *state = assert_cast<CommandLineFastParsingState*>(_state.get());
-
-  config *configuration = World::getInstance().getConfig();
-  configuration->FastParsing = state->newvalue;
-  if (configuration->FastParsing)
-    DoLog(0) && (Log() << Verbose(0) << "I won't parse trajectories." << endl);
-  else
-    DoLog(0) && (Log() << Verbose(0) << "I will parse trajectories." << endl);
-
-  return Action::state_ptr(_state);
+  return performUndo(_state);
 }