Changes in / [9784cf:4cf323d]

Files:

• src/Actions/MapOfActions.cpp (modified) (3 diffs)
• src/Actions/MapOfActions.hpp (modified) (1 diff)
• src/Actions/WorldAction/InputAction.cpp (modified) (1 diff)
• src/Parser/MpqcParser.cpp (modified) (1 diff)
• src/Parser/PcpParser.cpp (modified) (1 diff)
• src/Parser/PdbParser.cpp (modified) (2 diffs)
• src/Parser/TremoloParser.cpp (modified) (1 diff)
• src/UIElements/CommandLineUI/CommandLineDialog.cpp (modified) (1 diff)
• tests/regression/Simple_configuration/2/post/test.pdb (modified) (1 diff)

src/Actions/MapOfActions.cpp

@@ -191 +191 @@
   BV.zz = boost::lexical_cast<double>(components.at(5));
   v = boost::any(BoxValue(BV));
-}
-
-void validate(boost::any& v, const std::vector<std::string>& values, boost::filesystem::path *, int)
-{
-  boost::filesystem::path filename;
-  std::vector<std::string> components;
-
-  std::cout << "boost::filesystem::path validator used." << std::endl;
-
-  // split comma-separated values
-  if (values.size() != 1) {
-    cerr <<  "Not one file but " << components.size() << " given " << endl;
-    throw boost::program_options::validation_error("Unequal to one file given");
-  }
-  filename = values.at(0);
-  v = boost::any(boost::filesystem::path(filename));
 }
 
@@ -463 +447 @@
   TypeEnumMap[&typeid(bool)] = Boolean;
   TypeEnumMap[&typeid(int)] = Integer;
-  TypeEnumMap[&typeid(boost::filesystem::path)] = File;
   TypeEnumMap[&typeid(std::vector<int>)] = ListOfIntegers;
   TypeEnumMap[&typeid(double)] = Double;
@@ -1095 +1078 @@
               ;
             break;
-          case File:
-            ListRunner->second->add_options()
-              (getKeyAndShortForm(*OptionRunner).c_str(),
-                  po::value< boost::filesystem::path >(),
-                  getDescription(*OptionRunner).c_str())
-              ;
-            break;
           case Integer:
             ListRunner->second->add_options()

src/Actions/MapOfActions.hpp

@@ -138 +138 @@
   friend class MapOfActionsTest;
 public:
-  enum OptionTypes { None, Boolean, Integer, File, ListOfIntegers, Double, ListOfDoubles, String, ListOfStrings, Vector, ListOfVectors, Box, Molecule, ListOfMolecules, Atom, ListOfAtoms, Element, ListOfElements };
+  enum OptionTypes { None, Boolean, Integer, ListOfIntegers, Double, ListOfDoubles, String, ListOfStrings, Vector, ListOfVectors, Box, Molecule, ListOfMolecules, Atom, ListOfAtoms, Element, ListOfElements };
 
   // getter for the action descriptions and short forms

src/Actions/WorldAction/InputAction.cpp

@@ -49 +49 @@
 
   DoLog(0) && (Log() << Verbose(0) << "Config file given " << params.filename << "." << endl);
-  // using the filename as prefix for all parsers
-  std::string FilenameSuffix;
-  std::string FilenamePrefix;
-  if (params.filename.has_filename()) {
-    // get suffix
-    FilenameSuffix = params.filename.extension().substr(1); // remove the prefixed "."
-    FilenamePrefix = params.filename.stem();
-    DoLog(1) && (Log() << Verbose(1) << "Setting config file name prefix to " << FilenamePrefix << "." << endl);
-    FormatParserStorage::getInstance().SetOutputPrefixForAll(FilenamePrefix);
-  } else {
-    DoeLog(1) && (eLog() << Verbose(1) << "Input file does not have a suffix, cannot recognize format." << endl);
-    return Action::failure;
-  }
-
-  // parsing file if present
   if (!boost::filesystem::exists(params.filename)) {
     DoLog(1) && (Log() << Verbose(1) << "Specified config file " << params.filename << " not found." << endl);
-    // DONT FAIL: it's just empty and we use the name. // return Action::failure;
+    return Action::failure;
   } else {
     DoLog(1) && (Log() << Verbose(1) << "Specified config file found, parsing ... ");
+    if (params.filename.has_filename()) {
+      // get suffix
+      std::string FilenameSuffix = params.filename.extension();
+      std::string FilenamePrefix = params.filename.stem();
+      DoLog(1) && (Log() << Verbose(1) << "Setting config file name prefix to " << FilenamePrefix << "." << endl);
+      FormatParserStorage::getInstance().SetOutputPrefixForAll(FilenamePrefix);
 
-    // parse the file
-    test.open(params.filename);
-    FormatParserStorage::getInstance().get(test, FilenameSuffix);
-    test.close();
-
-    // set file name of last molecule
-    MoleculeList::const_iterator iter = World::getInstance().getMolecules()->ListOfMolecules.end();
-    iter--;
-    (*iter)->SetNameFromFilename(FilenamePrefix.c_str());
+      // parse the file
+      test.open(params.filename);
+      FormatParserStorage::getInstance().get(test, FilenameSuffix);
+      test.close();
+//
+//      // set mol to first active molecule
+//      if (molecules->ListOfMolecules.size() != 0) {
+//        for (MoleculeList::iterator ListRunner = molecules->ListOfMolecules.begin(); ListRunner != molecules->ListOfMolecules.end(); ListRunner++)
+//          if ((*ListRunner)->ActiveFlag) {
+//            mol = *ListRunner;
+//            break;
+//          }
+//      }
+//      if (mol == NULL) {
+//        mol = World::getInstance().createMolecule();
+//        mol->ActiveFlag = true;
+//        molecules->insert(mol);
+//      }
+//      mol->SetNameFromFilename(params.filename.substr(0,params.filename.find('.')).c_str());
+      return Action::success;
+    } else {
+      DoeLog(1) && (eLog() << Verbose(1) << "Input file does not have a suffix, cannot recognize format." << endl);
+      return Action::failure;
+    }
   }
-  return Action::success;
 }
 

src/Parser/MpqcParser.cpp

@@ -49 +49 @@
 void MpqcParser::load(istream *file)
 {
-  // TODO: MpqcParser::load implementation
-  ASSERT(false, "Not implemented yet");
+  DoeLog(0) && (Log() << Verbose(0) << "Not yet implemented" << endl) ;
 }
 

src/Parser/PcpParser.cpp

@@ -355 +355 @@
   // 3. parse the molecule in
   molecule *mol = World::getInstance().createMolecule();
+  MoleculeListClass *molecules = World::getInstance().getMolecules();
+  molecules->insert(mol);
+  LoadMolecule(mol, FileBuffer, World::getInstance().getPeriode(), FastParsing);
+  //mol->SetNameFromFilename(filename);
   mol->ActiveFlag = true;
-  // TODO: Remove the insertion into molecule when saving does not depend on them anymore. Also, remove molecule.hpp include
-  World::getInstance().getMolecules()->insert(mol);
-  LoadMolecule(mol, FileBuffer, World::getInstance().getPeriode(), FastParsing);
+  //MolList->insert(mol);
 
   // 4. dissect the molecule into connected subgraphs

src/Parser/PdbParser.cpp

@@ -74 +74 @@
  */
 void PdbParser::load(istream* file) {
-  // TODO: PdbParser::load implementation
-  ASSERT(false, "Not implemented yet");
 //  string line;
 //  string::size_type location;
@@ -134 +132 @@
       elementNo[Z] = (elementNo[Z]+1) % 100;   // confine to two digits
       const molecule *mol = (*atomIt)->getMolecule();
-      if (mol == NULL) {  // for homeless atoms, MolNo = -1 is reserved
-        MolNo = -1;
+      if (mol == NULL) {  // for homeless atoms, MolNo = 0 is reserved
+        MolNo = 0;
       } else {
         MolNo = mol->getId();

src/Parser/TremoloParser.cpp

@@ -92 +92 @@
   usedFields.clear();
   molecule *newmol = World::getInstance().createMolecule();
-  newmol->ActiveFlag = true;
-  // TODO: Remove the insertion into molecule when saving does not depend on them anymore. Also, remove molecule.hpp include
-  World::getInstance().getMolecules()->insert(newmol);
   while (file->good()) {
     std::getline(*file, line, '\n');

src/UIElements/CommandLineUI/CommandLineDialog.cpp

@@ -470 +470 @@
 bool CommandLineDialog::FileCommandLineQuery::handle() {
   if (CommandLineParser::getInstance().vm.count(getTitle())) {
-    tmp = CommandLineParser::getInstance().vm[getTitle()].as< boost::filesystem::path >();
+    tmp = CommandLineParser::getInstance().vm[getTitle()].as<string>();
     return true;
   } else {

tests/regression/Simple_configuration/2/post/test.pdb

@@ -1 +1 @@
 REMARK created by molecuilder on Fri Aug 27 12:18:33 2010
-ATOM      1  H01 non b   0        10.0    10.0    10.0   1.0   1.0         0 H 0
+ATOM      1  H01 non b   1        10.0    10.0    10.0   1.0   1.0         0 H 0
 END