Changeset 10707 for trunk/FACT++/src/triggerschedule.cc

Timestamp:

05/13/11 15:48:57 (14 years ago)

Author:

tbretz

Message:

Use the functionality of boost program_options better; implemented po_bool() as a special po_switch() case; added some comments.

File:

• trunk/FACT++/src/triggerschedule.cc (modified) (3 diffs)

trunk/FACT++/src/triggerschedule.cc

@@ -13 +13 @@
     config.add_options()
         ("dns",               var<string>("localhost"),  "Dim nameserver host name (Overwites DIM_DNS_NODE environment variable)")
-        ("schedule-database", var<string>(),  "Database name for scheduling preview")
+        ("schedule-database", var<string>()->required(), "Database name for scheduling preview")
         ;
 
@@ -64 +64 @@
         vm = conf.Parse(argc, argv);
     }
+#if BOOST_VERSION > 104000
+    catch (po::multiple_occurrences &e)
+    {
+        cout << "Error: " << e.what() << " of '" << e.get_option_name() << "' option." << endl;
+        cout << endl;
+        return -1;
+    }
+#endif
     catch (std::exception &e)
     {
-#if BOOST_VERSION > 104000
-        po::multiple_occurrences *MO = dynamic_cast<po::multiple_occurrences*>(&e);
-        if (MO)
-            cout << "Error: " << e.what() << " of '" << MO->get_option_name() << "' option." << endl;
-        else
-#endif
-            cout << "Error: " << e.what() << endl;
+        cout << "Error: " << e.what() << endl;
         cout << endl;
 
@@ -93 +95 @@
     }
 
-    if (!conf.Has("schedule-database"))
-    {
-        cout << "Please provide which database you want to use for scheduling using --schedule-database=<dbname>." << endl;
-        return -1;
-    }
-
     const char* dbname = conf.Get<string>("schedule-database").c_str();