Changes in src/builder.cpp [99fcaf:4eb4fe]

File:

• src/builder.cpp (modified) (5 diffs)

src/builder.cpp

@@ -1494 +1494 @@
  */
 static int ParseCommandLineOptions(int argc, char **argv, MoleculeListClass *&molecules, periodentafel *&periode,
-                                   config& configuration, char *&ConfigFileName, list<int> &ArgcList)
+                                   config& configuration, char *&ConfigFileName, set<int> &ArgcList)
 {
   Vector x,y,z,n;  // coordinates for absolute point in cell volume
@@ -1525 +1525 @@
           case 'H':
           case '?':
-            ArgcList.push_back(argptr-1);
+            ArgcList.insert(argptr-1);
             return(1);
             break;
           case 'v':
-            ArgcList.push_back(argptr-1);
+            ArgcList.insert(argptr-1);
             return(1);
             break;
           case 'V':
-            ArgcList.push_back(argptr-1);
-            ArgcList.push_back(argptr);
+            ArgcList.insert(argptr-1);
+            ArgcList.insert(argptr);
             argptr++;
             break;
@@ -2499 +2499 @@
     char **Arguments = NULL;
     int ArgcSize = 0;
-    int ExitFlag = 0;
     bool ArgumentsCopied = false;
 
+    // print version check whether arguments are present at all
     cout << ESPACKVersion << endl;
+    if (argc < 2) {
+      cout << "Obtain help with " << argv[0] << " -h." << endl;
+      return(1);
+    }
+
 
     setVerbosity(0);
@@ -2510 +2515 @@
     // Parse command line options and if present create respective UI
     {
-      list<int> ArgcList;
-      ArgcList.push_back(0); // push back program!
-      ArgcList.push_back(1); // push back config file name
+      set<int> ArgcList;
+      ArgcList.insert(0); // push back program!
+      ArgcList.insert(1); // push back config file name
       char ConfigFileName[MAXSTRINGSIZE];
       // handle arguments by ParseCommandLineOptions()
-      ExitFlag = ParseCommandLineOptions(argc,argv,World::getInstance().getMolecules(),World::getInstance().getPeriode(),*World::getInstance().getConfig(), (char *&)ConfigFileName, ArgcList);
+      ParseCommandLineOptions(argc,argv,World::getInstance().getMolecules(),World::getInstance().getPeriode(),*World::getInstance().getConfig(), (char *&)ConfigFileName, ArgcList);
       // copy all remaining arguments to a new argv
       Arguments = Malloc<char *>(ArgcList.size(), "main - **Arguments");
       cout << "The following arguments are handled by CommandLineParser: ";
-      for (list<int>::iterator ArgcRunner = ArgcList.begin(); ArgcRunner != ArgcList.end(); ++ArgcRunner) {
+      for (set<int>::iterator ArgcRunner = ArgcList.begin(); ArgcRunner != ArgcList.end(); ++ArgcRunner) {
         Arguments[ArgcSize] = Malloc<char>(strlen(argv[*ArgcRunner])+2, "main - *Arguments[]");
         strcpy(Arguments[ArgcSize], argv[*ArgcRunner]);
@@ -2560 +2565 @@
   cleanUp();
   Memory::getState();
-  return (ExitFlag == 1 ? 0 : ExitFlag);
+  return (0);
 }