[svn] commit: r63 - in /experiments/jelte-configuration: config_manager.cc config_obj.cc

BIND 10 source code commits bind10-changes at lists.isc.org
Tue Oct 6 08:58:23 UTC 2009


Author: jelte
Date: Tue Oct  6 08:58:23 2009
New Revision: 63

Log:
catch by reference

Modified:
    experiments/jelte-configuration/config_manager.cc
    experiments/jelte-configuration/config_obj.cc

Modified: experiments/jelte-configuration/config_manager.cc
==============================================================================
--- experiments/jelte-configuration/config_manager.cc (original)
+++ experiments/jelte-configuration/config_manager.cc Tue Oct  6 08:58:23 2009
@@ -33,7 +33,7 @@
 {
     try {
         config = new Config(filename);
-    } catch (ConfigError e) {
+    } catch (ConfigError& e) {
         delete config;
         std::string err;
         err = "Error reading " + filename + ": " + e.what();
@@ -46,7 +46,7 @@
 {
     try {
         config->write_file(filename);
-    } catch (ConfigError e) {
+    } catch (ConfigError& e) {
         std::string err;
         err = "Error writing " + filename + ": " + e.what();
         throw ConfigManagerError(err);
@@ -100,7 +100,7 @@
             //config_part->set_value("@name", "asdf.com");
             cout << "Selected zone: " << endl;
             config_part->write_stream(std::cout);
-        } catch (ConfigError ce) {
+        } catch (ConfigError& ce) {
             cout << "Caught ConfigError: " << ce.what() << endl;
         }
 
@@ -133,7 +133,7 @@
             config->set_config_part("/module[@name=recursive]/zones/zone[@name=theo.com]", config_part);
 
             delete config_part;
-        } catch (ConfigError ce) {
+        } catch (ConfigError& ce) {
             cout << "Caught ConfigError: " << ce.what() << endl;
         }
 

Modified: experiments/jelte-configuration/config_obj.cc
==============================================================================
--- experiments/jelte-configuration/config_obj.cc (original)
+++ experiments/jelte-configuration/config_obj.cc Tue Oct  6 08:58:23 2009
@@ -251,7 +251,7 @@
         try {
             config_part->node = find_sub_node(node, identifier)->cloneNode(true);
             return config_part;
-        } catch (ConfigError ce) {
+        } catch (ConfigError& ce) {
             delete config_part;
             throw ce;
         }




More information about the bind10-changes mailing list