Commit b5696786 authored by Marcin Siodelski's avatar Marcin Siodelski
Browse files

[64-client-class-cmds-hook] Better description of dependent_class parameter.

This is in the ClientClassDef::dependOnClass. Review comment.
parent 08d650e4
......@@ -287,14 +287,14 @@ ClientClassDictionary::getClasses() const {
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 {
......@@ -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.
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment