Browse Source

[3341] Addressed review comments.

Improved doxygen commentary for new methods.
Thomas Markwalder 11 years ago
parent
commit
e724186c63
2 changed files with 12 additions and 2 deletions
  1. 1 1
      src/bin/d2/d2_cfg_mgr.cc
  2. 11 1
      src/bin/d2/d2_cfg_mgr.h

+ 1 - 1
src/bin/d2/d2_cfg_mgr.cc

@@ -1,4 +1,4 @@
-// Copyright (C) 2013 Internet Systems Consortium, Inc. ("ISC")
+// Copyright (C) 2014 Internet Systems Consortium, Inc. ("ISC")
 //
 // Permission to use, copy, modify, and/or distribute this software for any
 // purpose with or without fee is hereby granted, provided that the above

+ 11 - 1
src/bin/d2/d2_cfg_mgr.h

@@ -1,4 +1,4 @@
-// Copyright (C) 2013 Internet Systems Consortium, Inc. ("ISC")
+// Copyright (C) 2014 Internet Systems Consortium, Inc. ("ISC")
 //
 // Permission to use, copy, modify, and/or distribute this software for any
 // purpose with or without fee is hereby granted, provided that the above
@@ -128,11 +128,21 @@ public:
 
     /// @brief Returns whether not forward updates are enabled.
     ///
+    /// This method currently uses the presence or absence of Foward DDNS
+    /// Domains to determine if forward updates are enabled or disabled.
+    /// @todo This could be expanded to include the check of a configurable
+    /// boolean value.
+    ///
     /// @return true if forward updates are enabled, false otherwise.
     bool forwardUpdatesEnabled();
 
     /// @brief Returns whether not reverse updates are enabled.
     ///
+    /// This method currently uses the presence or absence of Reverse DDNS
+    /// Domains to determine if reverse updates are enabled or disabled.
+    /// @todo This could be expanded to include the check of a configurable
+    /// boolean value.
+    ///
     /// @return true if reverse updates are enabled, false otherwise.
     bool reverseUpdatesEnabled();