Simplify DocIterator

[lyx.git] / src / BranchList.cpp

diff --git a/src/BranchList.cpp b/src/BranchList.cpp
index 9c0078b316ef52afb4ae7df2d85e73a8c0da929a..01135526e64b72f077949aa776e2f6261e30fc98 100644 (file)
--- a/src/BranchList.cpp
+++ b/src/BranchList.cpp
@@ -25,26 +25,6 @@ using namespace std;
 
 namespace lyx {
 
-namespace {
-
-class BranchNamesEqual : public std::unary_function<Branch, bool>
-{
-public:
-       BranchNamesEqual(docstring const & name)
-               : name_(name)
-       {}
-
-       bool operator()(Branch const & branch) const
-       {
-               return branch.branch() == name_;
-       }
-private:
-       docstring name_;
-};
-
-} // namespace
-
-
 Branch::Branch()
        : selected_(false), filenameSuffix_(false)
 {
@@ -122,19 +102,29 @@ void Branch::setColor(string const & str)
 }
 
 
+namespace {
+
+std::function<bool (Branch const &)> BranchNameIs(docstring const & d)
+{
+       return [d](Branch const & b){ return b.branch() == d; };
+}
+
+} // namespace
+
+
 Branch * BranchList::find(docstring const & name)
 {
        List::iterator it =
-               find_if(list_.begin(), list_.end(), BranchNamesEqual(name));
-       return it == list_.end() ? 0 : &*it;
+               find_if(list_.begin(), list_.end(), BranchNameIs(name));
+       return it == list_.end() ? nullptr : &*it;
 }
 
 
 Branch const * BranchList::find(docstring const & name) const
 {
        List::const_iterator it =
-               find_if(list_.begin(), list_.end(), BranchNamesEqual(name));
-       return it == list_.end() ? 0 : &*it;
+               find_if(list_.begin(), list_.end(), BranchNameIs(name));
+       return it == list_.end() ? nullptr : &*it;
 }
 
 
@@ -150,9 +140,7 @@ bool BranchList::add(docstring const & s)
                else
                        name = s.substr(i, j - i);
                // Is this name already in the list?
-               bool const already =
-                       find_if(list_.begin(), list_.end(),
-                                        BranchNamesEqual(name)) != list_.end();
+               bool const already = find(name);
                if (!already) {
                        added = true;
                        Branch br;
@@ -172,7 +160,7 @@ bool BranchList::add(docstring const & s)
 bool BranchList::remove(docstring const & s)
 {
        size_t const size = list_.size();
-       list_.remove_if(BranchNamesEqual(s));
+       list_.remove_if(BranchNameIs(s));
        return size != list_.size();
 }
 
@@ -182,8 +170,7 @@ bool BranchList::rename(docstring const & oldname,
 {
        if (newname.empty())
                return false;
-       if (find_if(list_.begin(), list_.end(),
-                       BranchNamesEqual(newname)) != list_.end()) {
+       if (find(newname)) {
                // new name already taken
                if (merge)
                      return remove(oldname);