Changeset 0e348e for molecuilder/src

Timestamp:

May 16, 2008, 1:35:43 PM (17 years ago)

Author:

Frederik Heber <heber@…>

Children:

e9c14d

Parents:

cc6dfb

Message:

main(): typo in Usage message, BUGFIX - command line arguments did not end in exit and flag renamed to ExitFlag

BUGFIX - problem was flag > 1 instead of flag >= 1 check

File:

• molecuilder/src/builder.cpp (modified) (10 diffs)

molecuilder/src/builder.cpp

@@ -717 +717 @@
   char *ConfigFileName = NULL;
   char *ElementsFileName = NULL;
-  int flag = 0;
+  int ExitFlag = 0;
   int Z;
   int j, axis, count, faktor;
@@ -746 +746 @@
             cout << "\t-c x1 x2 x3\tCenter atoms in domain with a minimum distance to boundary of (x1,x2,x3)." << endl;
             cout << "\t-e <file>\tSets the element database to be parsed from this file (default: elements.db in same dir as " << argv[0] << ")." << endl;
-            cout << "\t-e <dist> <order>\tFragments the molecule in BOSSANOVA manner and stores config files in same dir as config." << endl;
+            cout << "\t-f <dist> <order>\tFragments the molecule in BOSSANOVA manner and stores config files in same dir as config." << endl;
             cout << "\t-h/-H/-?\tGive this help screen." << endl;
             cout << "\t-p <file>\tParse given xyz file and create raw config file from it." << endl;
@@ -832 +832 @@
           switch(argv[argptr-1][1]) {
             case 'p':
-              flag = 1;
+              ExitFlag = 1;
               cout << Verbose(1) << "Parsing xyz file for new atoms." << endl;
               if (!mol->AddXYZFile(argv[argptr++]))
@@ -848 +848 @@
             switch(argv[argptr-1][1]) {
               case 't':
-                flag = 1;
+                ExitFlag = 1;
                 cout << Verbose(1) << "Translating all ions to new origin." << endl;
                 for (int i=0;i<3;i++)
@@ -856 +856 @@
                 break;
               case 'a':
-                flag = 1;
+                ExitFlag = 1;
                 cout << Verbose(1) << "Adding new atom." << endl;
                 first = new atom;
@@ -873 +873 @@
                 break;
               case 's':
-                flag = 1;
+                ExitFlag = 1;
                 j = -1;
                 cout << Verbose(1) << "Scaling all ion positions by factor." << endl;
@@ -894 +894 @@
                 break;
               case 'c':
-                flag = 1;
+                ExitFlag = 1;
                 j = -1;
                 cout << Verbose(1) << "Centering atoms in config file within given additional boundary." << endl;
@@ -911 +911 @@
                 break;
               case 'r':
-                flag = 1;
+                ExitFlag = 1;
                 cout << Verbose(1) << "Converting config file from supposed old to new syntax." << endl;
                 break;
               case 'f':
-                if (flag ==0) flag = 2;  // only set if not already by other command line switch
+                if (ExitFlag ==0) ExitFlag = 2;  // only set if not already by other command line switch
                 cout << "Fragmenting molecule with bond distance " << argv[argptr] << " angstroem, order of " << argv[argptr+1] << "." << endl;
                 if (argc >= argptr+2) {
@@ -940 +940 @@
           } else argptr++;
         } while (argptr < argc);
-    if (flag == 1) // 1 means save and exit
+    if (ExitFlag == 1) // 1 means save and exit
       SaveConfig(ConfigFileName, &configuration, periode, mol);
-    if (flag > 1) { // 2 means just exit
+    if (ExitFlag >= 1) { // 2 means just exit
       delete(mol);
       delete(periode);
@@ -1102 +1102 @@
         
       case 'c': // unit scaling of the metric 
+       cout << Verbose(0) << "Angstroem -> Bohrradius: 1.8897261\t\tBohrradius -> Angstroem: 0.52917721" << endl;
          cout << Verbose(0) << "Enter three factors: ";
        factor = (double *) Malloc(sizeof(double)*NDIM, "main: *factor");