]> git.lyx.org Git - lyx.git/blobdiff - src/bufferlist.C
minimal effort implementation of:
[lyx.git] / src / bufferlist.C
index 174dd2cc6e4ff1b75c7563962ac7d5f5d47fd577..861bb6fb43bab197ca981a2d719775cadd28f3f5 100644 (file)
 #include "bufferparams.h"
 #include "debug.h"
 #include "gettext.h"
-#include "lastfiles.h"
+#include "session.h"
 #include "lyx_cb.h"
 #include "lyx_main.h"
 #include "output_latex.h"
 #include "paragraph.h"
-#include "ParagraphList_fwd.h"
+#include "ParagraphList.h"
 
 #include "frontends/Alert.h"
 
 #include "support/filetools.h"
-#include "support/os.h"
+#include "support/package.h"
 
 #include <boost/bind.hpp>
 
 #include <algorithm>
 #include <functional>
 
-using lyx::support::AddName;
+using lyx::support::addName;
 using lyx::support::bformat;
-using lyx::support::MakeAbsPath;
-using lyx::support::MakeDisplayPath;
-using lyx::support::OnlyFilename;
+using lyx::support::makeAbsPath;
+using lyx::support::makeDisplayPath;
+using lyx::support::onlyFilename;
 using lyx::support::removeAutosaveFile;
+using lyx::support::package;
 using lyx::support::prefixIs;
 
 using boost::bind;
@@ -55,7 +56,6 @@ using std::vector;
 using std::back_inserter;
 using std::transform;
 
-namespace os = lyx::support::os;
 
 BufferList::BufferList()
 {}
@@ -69,14 +69,17 @@ bool BufferList::empty() const
 
 bool BufferList::quitWriteBuffer(Buffer * buf)
 {
+       BOOST_ASSERT(buf);
+
        string file;
        if (buf->isUnnamed())
-               file = OnlyFilename(buf->fileName());
+               file = onlyFilename(buf->fileName());
        else
-               file = MakeDisplayPath(buf->fileName(), 30);
+               file = makeDisplayPath(buf->fileName(), 30);
 
-       string text = bformat(_("The document %1$s has unsaved changes.\n\n"
-               "Do you want to save the document or discard the changes?"), file);
+       string const text =
+               bformat(_("The document %1$s has unsaved changes.\n\n"
+                         "Do you want to save the document or discard the changes?"), file);
        int const ret = Alert::prompt(_("Save changed document?"),
                text, 0, 2, _("&Save"), _("&Discard"), _("&Cancel"));
 
@@ -87,9 +90,9 @@ bool BufferList::quitWriteBuffer(Buffer * buf)
                bool succeeded;
 
                if (buf->isUnnamed())
-                       succeeded = WriteAs(buf);
+                       succeeded = writeAs(buf);
                else
-                       succeeded = MenuWrite(buf);
+                       succeeded = menuWrite(buf);
 
                if (!succeeded)
                        return false;
@@ -119,6 +122,15 @@ bool BufferList::quitWriteAll()
                if (!quitWriteBuffer(*it))
                        return false;
        }
+       // now, all buffers have been written sucessfully
+       // save file names to .lyx/session
+       it = bstore.begin();
+       for (; it != end; ++it) {
+               // if master/slave are both open, do not save slave since it 
+               // will be automatically loaded when the master is loaded
+               if ((*it)->getMasterBuffer() == (*it))
+                       LyX::ref().session().addLastOpenedFile((*it)->fileName());
+       }
 
        return true;
 }
@@ -127,16 +139,18 @@ bool BufferList::quitWriteAll()
 void BufferList::release(Buffer * buf)
 {
        BOOST_ASSERT(buf);
-       BufferStorage::iterator it = find(bstore.begin(), bstore.end(), buf);
+       BufferStorage::iterator const it =
+               find(bstore.begin(), bstore.end(), buf);
        if (it != bstore.end()) {
                Buffer * tmp = (*it);
+               BOOST_ASSERT(tmp);
                bstore.erase(it);
                delete tmp;
        }
 }
 
 
-Buffer * BufferList::newBuffer(string const & s, bool ronly)
+Buffer * BufferList::newBuffer(string const & s, bool const ronly)
 {
        auto_ptr<Buffer> tmpbuf(new Buffer(s, ronly));
        tmpbuf->params().useClassDefaults();
@@ -155,39 +169,37 @@ void BufferList::closeAll()
 }
 
 
-bool BufferList::close(Buffer * buf, bool ask)
+bool BufferList::close(Buffer * buf, bool const ask)
 {
        BOOST_ASSERT(buf);
 
-       // FIXME: is the quitting check still necessary ?
-       if (!ask || buf->isClean() || quitting || buf->paragraphs().empty()) {
+       if (!ask || buf->isClean() || buf->paragraphs().empty()) {
                release(buf);
                return true;
        }
 
        string fname;
        if (buf->isUnnamed())
-               fname = OnlyFilename(buf->fileName());
+               fname = onlyFilename(buf->fileName());
        else
-               fname = MakeDisplayPath(buf->fileName(), 30);
+               fname = makeDisplayPath(buf->fileName(), 30);
 
-       string text = bformat(_("The document %1$s has unsaved changes.\n\n"
-               "Do you want to save the document or discard the changes?"), fname);
+       string const text =
+               bformat(_("The document %1$s has unsaved changes.\n\n"
+                         "Do you want to save the document or discard the changes?"), fname);
        int const ret = Alert::prompt(_("Save changed document?"),
                text, 0, 2, _("&Save"), _("&Discard"), _("&Cancel"));
 
        if (ret == 0) {
                if (buf->isUnnamed()) {
-                       if (!WriteAs(buf))
+                       if (!writeAs(buf))
                                return false;
-               } else if (buf->save()) {
-                       LyX::ref().lastfiles().newFile(buf->fileName());
-               } else {
+               } else if (!menuWrite(buf))
                        return false;
-               }
-       } else if (ret == 2) {
+               else
+                       return false;
+       } else if (ret == 2)
                return false;
-       }
 
        if (buf->isUnnamed()) {
                removeAutosaveFile(buf->fileName());
@@ -216,7 +228,7 @@ Buffer * BufferList::first()
 }
 
 
-Buffer * BufferList::getBuffer(unsigned int choice)
+Buffer * BufferList::getBuffer(unsigned int const choice)
 {
        if (choice >= bstore.size())
                return 0;
@@ -226,6 +238,8 @@ Buffer * BufferList::getBuffer(unsigned int choice)
 
 Buffer * BufferList::next(Buffer const * buf) const
 {
+       BOOST_ASSERT(buf);
+
        if (bstore.empty())
                return 0;
        BufferStorage::const_iterator it = find(bstore.begin(),
@@ -241,6 +255,8 @@ Buffer * BufferList::next(Buffer const * buf) const
 
 Buffer * BufferList::previous(Buffer const * buf) const
 {
+       BOOST_ASSERT(buf);
+
        if (bstore.empty())
                return 0;
        BufferStorage::const_iterator it = find(bstore.begin(),
@@ -280,17 +296,16 @@ void BufferList::emergencyWriteAll()
 
 void BufferList::emergencyWrite(Buffer * buf)
 {
-       // assert(buf) // this is not good since C assert takes an int
-                      // and a pointer is a long (JMarc)
-       assert(buf != 0); // use c assert to avoid a loop
-
+       // Use ::assert to avoid a loop, BOOST_ASSERT ends up calling ::assert
+       // compare with 0 to avoid pointer/interger comparison
+       assert(buf != 0);
 
        // No need to save if the buffer has not changed.
        if (buf->isClean())
                return;
 
        string const doc = buf->isUnnamed()
-               ? OnlyFilename(buf->fileName()) : buf->fileName();
+               ? onlyFilename(buf->fileName()) : buf->fileName();
 
        lyxerr << bformat(_("LyX: Attempting to save document %1$s"), doc) << endl;
 
@@ -310,7 +325,7 @@ void BufferList::emergencyWrite(Buffer * buf)
        }
 
        // 2) In HOME directory.
-       string s = AddName(os::homepath(), buf->fileName());
+       string s = addName(package().home_dir(), buf->fileName());
        s += ".emergency";
        lyxerr << ' ' << s << endl;
        if (buf->writeFile(s)) {
@@ -324,7 +339,7 @@ void BufferList::emergencyWrite(Buffer * buf)
        // 3) In "/tmp" directory.
        // MakeAbsPath to prepend the current
        // drive letter on OS/2
-       s = AddName(MakeAbsPath("/tmp/"), buf->fileName());
+       s = addName(package().temp_dir(), buf->fileName());
        s += ".emergency";
        lyxerr << ' ' << s << endl;
        if (buf->writeFile(s)) {