Lars Gullik Bjønnes wrote:

> bool BranchList::remove(string const & s)
>  {
>          List::size_type const size = list.size();
> -        list.remove_if(bind2nd(match(), s));
> +        list.remove_if(bind(equal_to<string>(),
> +                            bind(&Branch::getBranch, _1),
> +                            s));
>          return size != list.size();
>  }

Lars! Be nice to us! Add comments explaining what this stuff is 
equivalent to. If you're not careful, your fellow developers will 
start to view you as the evil genius who obfuscates their perfectly 
reasonable code and then laughs at their confusion, "Mu hah hah hah!"

If you're going to go down this boostification-for-the-sake-of-it 
route, then please try and take us all with you.

Would boost::lambda help here?

-- 
Angus

Reply via email to