Remove obsolete (and false) comment.

[lyx.git] / src / BranchList.cpp

diff --git a/src/BranchList.cpp b/src/BranchList.cpp
index 02f8b8688d4d377976588f7d58ec7fa7c0bdfca6..9c0078b316ef52afb4ae7df2d85e73a8c0da929a 100644 (file)
--- a/src/BranchList.cpp
+++ b/src/BranchList.cpp
@@ -4,6 +4,7 @@
  * Licence details can be found in the file COPYING.
  *
  * \author Martin Vermeer
+ * \author Jürgen Spitzmüller
  *
  * Full author contact details are available in file CREDITS.
  */
@@ -15,6 +16,8 @@
 
 #include "frontends/Application.h"
 
+#include "support/lstrings.h"
+
 #include <algorithm>
 
 using namespace std;
@@ -27,7 +30,9 @@ namespace {
 class BranchNamesEqual : public std::unary_function<Branch, bool>
 {
 public:
-       BranchNamesEqual(docstring const & name) : name_(name) {}
+       BranchNamesEqual(docstring const & name)
+               : name_(name)
+       {}
 
        bool operator()(Branch const & branch) const
        {
@@ -36,14 +41,18 @@ public:
 private:
        docstring name_;
 };
-}
+
+} // namespace
 
 
-Branch::Branch() : selected_(false)
+Branch::Branch()
+       : selected_(false), filenameSuffix_(false)
 {
        // no theApp() with command line export
        if (theApp())
                theApp()->getRgbColor(Color_background, color_);
+       else
+               frontend::Application::getRgbColorUncached(Color_background, color_);
 }
 
 
@@ -74,6 +83,18 @@ bool Branch::setSelected(bool b)
 }
 
 
+bool Branch::hasFileNameSuffix() const
+{
+       return filenameSuffix_;
+}
+
+
+void Branch::setFileNameSuffix(bool b)
+{
+       filenameSuffix_ = b;
+}
+
+
 RGBColor const & Branch::color() const
 {
        return color_;
@@ -90,25 +111,30 @@ void Branch::setColor(string const & str)
 {
        if (str.size() == 7 && str[0] == '#')
                color_ = rgbFromHexName(str);
-       else
+       else {
                // no color set or invalid color - use normal background
-               theApp()->getRgbColor(Color_background, color_);
+               // no theApp() with command line export
+               if (theApp())
+                       theApp()->getRgbColor(Color_background, color_);
+               else
+                       frontend::Application::getRgbColorUncached(Color_background, color_);
+       }
 }
 
 
 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(), BranchNamesEqual(name));
+       return it == list_.end() ? 0 : &*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(), BranchNamesEqual(name));
+       return it == list_.end() ? 0 : &*it;
 }
 
 
@@ -125,14 +151,15 @@ bool BranchList::add(docstring const & s)
                        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();
+                       find_if(list_.begin(), list_.end(),
+                                        BranchNamesEqual(name)) != list_.end();
                if (!already) {
                        added = true;
                        Branch br;
                        br.setBranch(name);
                        br.setSelected(false);
-                       list.push_back(br);
+                       br.setFileNameSuffix(false);
+                       list_.push_back(br);
                }
                if (j == docstring::npos)
                        break;
@@ -144,9 +171,9 @@ bool BranchList::add(docstring const & s)
 
 bool BranchList::remove(docstring const & s)
 {
-       size_t const size = list.size();
-       list.remove_if(BranchNamesEqual(s));
-       return size != list.size();
+       size_t const size = list_.size();
+       list_.remove_if(BranchNamesEqual(s));
+       return size != list_.size();
 }
 
 
@@ -155,8 +182,8 @@ bool BranchList::rename(docstring const & oldname,
 {
        if (newname.empty())
                return false;
-       if (find_if(list.begin(), list.end(),
-                   BranchNamesEqual(newname)) != list.end()) {
+       if (find_if(list_.begin(), list_.end(),
+                       BranchNamesEqual(newname)) != list_.end()) {
                // new name already taken
                if (merge)
                      return remove(oldname);
@@ -171,4 +198,14 @@ bool BranchList::rename(docstring const & oldname,
 }
 
 
+docstring BranchList::getFileNameSuffix() const
+{
+       docstring result;
+       for (auto const & br : list_) {
+               if (br.isSelected() && br.hasFileNameSuffix())
+                       result += "-" + br.branch();
+       }
+       return support::subst(result, from_ascii("/"), from_ascii("_"));
+}
+
 } // namespace lyx