]> git.ipfire.org Git - thirdparty/kea.git/commitdiff
[64-client-class-cmds-hook] Better description of dependent_class parameter.
authorMarcin Siodelski <marcin@isc.org>
Fri, 26 Oct 2018 09:51:02 +0000 (11:51 +0200)
committerMarcin Siodelski <marcin@isc.org>
Mon, 5 Nov 2018 16:38:48 +0000 (11:38 -0500)
This is in the ClientClassDef::dependOnClass. Review comment.

src/lib/dhcpsrv/client_class_def.cc
src/lib/dhcpsrv/client_class_def.h

index aeabd96d0f236ddd452a237fcf6af6f694e392c1..2a9a52098a74e949a5f29e96b2f6dd340cb0e656 100644 (file)
@@ -287,14 +287,14 @@ ClientClassDictionary::getClasses() const {
 
 bool
 ClientClassDictionary::dependOnClass(const std::string& name,
-                                     std::string& depend) const {
+                                     std::string& dependent_class) const {
     // Skip previous classes as they should not depend on name.
     bool found = false;
     for (ClientClassDefList::iterator this_class = list_->begin();
          this_class != list_->end(); ++this_class) {
         if (found) {
             if ((*this_class)->dependOnClass(name)) {
-                depend = (*this_class)->getName();
+                dependent_class = (*this_class)->getName();
                 return (true);
             }
         } else {
index 111458aedcee185225d477cf3aa1fd8aef031948..94e03c2c56af907fc173b296c8bbae13c804ba02 100644 (file)
@@ -333,10 +333,11 @@ public:
     /// @brief Checks direct dependency.
     ///
     /// @param name The client class name.
-    /// @param depend Set to the name of the first depending class.
+    /// @param [out] dependent_class Reference to a variable where the
+    /// name of the first class depending on the checked class is set.
     ///
     /// @return true if a definition depends on the class name, false if none.
-    bool dependOnClass(const std::string& name, std::string& depend) const;
+    bool dependOnClass(const std::string& name, std::string& dependent_class) const;
 
     /// @brief Compares two @c ClientClassDictionary objects for equality.
     ///