Re: Patch [Re: BranchList class for comment]

[Alfredo Braunstein]

Martin Vermeer wrote:

+void BranchList::setColor(string const & s, string const & val)
+{
+       List::iterator it = list.begin();
+       List::iterator end = list.end();
+       for (; it != end; it++) {
+               if (s.find(it->getBranch(), 0) != string::npos) {
+                       it->setColor(val);


Sorry to insist Martin, but you should add a return here; otherwise every
call to BranchList::setColor will Assert...


+               }
+       }
+       Assert(false);
+}


Regards, Alfredo