X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2FBufferList.cpp;h=1a538d413f0ab1178e398b1fb70bb1cfb6b0caba;hb=ece5b8818d011cdf1e2d49fa06a3d4ffce5eb23e;hp=1bee03276b0cbdcc92230e6eb622e8a2825fff28;hpb=1447a7bf98fb691e9979fd3c7689adba4ed3f421;p=lyx.git diff --git a/src/BufferList.cpp b/src/BufferList.cpp index 1bee03276b..1a538d413f 100644 --- a/src/BufferList.cpp +++ b/src/BufferList.cpp @@ -3,7 +3,7 @@ * This file is part of LyX, the document processor. * Licence details can be found in the file COPYING. * - * \author Lars Gullik Bjønnes + * \author Lars Gullik Bjønnes * * Full author contact details are available in file CREDITS. */ @@ -15,26 +15,22 @@ #include "Author.h" #include "Buffer.h" #include "BufferParams.h" -#include "Session.h" -#include "LyX.h" -#include "output_latex.h" -#include "ParagraphList.h" +#include "OutputParams.h" #include "frontends/alert.h" #include "support/ExceptionMessage.h" #include "support/debug.h" +#include "support/FileName.h" +#include "support/FileNameList.h" #include "support/filetools.h" -#include "support/gettext.h" #include "support/lstrings.h" -#include "support/Package.h" -#include +#include "support/lassert.h" #include -#include - -using boost::bind; +#include +#include using namespace std; using namespace lyx::support; @@ -48,6 +44,13 @@ BufferList::BufferList() {} +BufferList::~BufferList() +{ + for (Buffer * buf : binternal) + delete buf; +} + + bool BufferList::empty() const { return bstore.empty(); @@ -80,34 +83,62 @@ BufferList::const_iterator BufferList::end() const void BufferList::release(Buffer * buf) { - BOOST_ASSERT(buf); + // We may leak here, but we probably do not need to + // shut down. + LASSERT(buf, return); BufferStorage::iterator const it = find(bstore.begin(), bstore.end(), buf); if (it != bstore.end()) { + Buffer const * parent = buf->parent(); Buffer * tmp = (*it); - BOOST_ASSERT(tmp); bstore.erase(it); + LASSERT(tmp, return); delete tmp; + if (parent) + // If this was a child, update the parent's buffer + // to avoid crashes due to dangling pointers (bug 9979) + parent->updateBuffer(); + } +} + + +Buffer * BufferList::newInternalBuffer(string const & s) +{ + Buffer * const buf = createNewBuffer(s); + if (buf) { + buf->setInternal(true); + binternal.push_back(buf); } + return buf; } -Buffer * BufferList::newBuffer(string const & s, bool const ronly) +Buffer * BufferList::newBuffer(string const & s) { - auto_ptr tmpbuf; - try { tmpbuf.reset(new Buffer(s, ronly)); + Buffer * const buf = createNewBuffer(s); + if (buf) { + LYXERR(Debug::INFO, "Assigning to buffer " << bstore.size()); + bstore.push_back(buf); + } + return buf; +} + + +Buffer * BufferList::createNewBuffer(string const & s) +{ + unique_ptr tmpbuf; + try { + tmpbuf = make_unique(s); } catch (ExceptionMessage const & message) { if (message.type_ == ErrorException) { Alert::error(message.title_, message.details_); exit(1); } else if (message.type_ == WarningException) { Alert::warning(message.title_, message.details_); - return 0; + return nullptr; } } tmpbuf->params().useClassDefaults(); - LYXERR(Debug::INFO, "Assigning to buffer " << bstore.size()); - bstore.push_back(tmpbuf.get()); return tmpbuf.release(); } @@ -119,12 +150,11 @@ void BufferList::closeAll() } -vector const BufferList::getFileNames() const +FileNameList BufferList::fileNames() const { - vector nvec; - transform(bstore.begin(), bstore.end(), - back_inserter(nvec), - boost::bind(&Buffer::absFileName, _1)); + FileNameList nvec; + for (Buffer const * buf : bstore) + nvec.push_back(buf->fileName()); return nvec; } @@ -132,7 +162,7 @@ vector const BufferList::getFileNames() const Buffer * BufferList::first() { if (bstore.empty()) - return 0; + return nullptr; return bstore.front(); } @@ -140,7 +170,7 @@ Buffer * BufferList::first() Buffer * BufferList::last() { if (bstore.empty()) - return 0; + return nullptr; return bstore.back(); } @@ -148,55 +178,54 @@ Buffer * BufferList::last() Buffer * BufferList::getBuffer(unsigned int choice) { if (choice >= bstore.size()) - return 0; + return nullptr; return bstore[choice]; } Buffer * BufferList::next(Buffer const * buf) const { - BOOST_ASSERT(buf); + // Something is wrong, but we can probably survive it. + LASSERT(buf, return nullptr); if (bstore.empty()) - return 0; - BufferStorage::const_iterator it = find(bstore.begin(), - bstore.end(), buf); - BOOST_ASSERT(it != bstore.end()); + return nullptr; + BufferStorage::const_iterator it = + find(bstore.begin(), bstore.end(), buf); + LASSERT(it != bstore.end(), return nullptr); ++it; - if (it == bstore.end()) - return bstore.front(); - else - return *it; + Buffer * nextbuf = (it == bstore.end()) ? bstore.front() : *it; + return nextbuf; } Buffer * BufferList::previous(Buffer const * buf) const { - BOOST_ASSERT(buf); + // Something is wrong, but we can probably survive it. + LASSERT(buf, return nullptr); if (bstore.empty()) - return 0; - BufferStorage::const_iterator it = find(bstore.begin(), - bstore.end(), buf); - BOOST_ASSERT(it != bstore.end()); - if (it == bstore.begin()) - return bstore.back(); - else - return *(it - 1); + return nullptr; + BufferStorage::const_iterator it = + find(bstore.begin(), bstore.end(), buf); + LASSERT(it != bstore.end(), return nullptr); + + Buffer * previousbuf = (it == bstore.begin()) ? bstore.back() : *(it - 1); + return previousbuf; } void BufferList::updateIncludedTeXfiles(string const & masterTmpDir, - OutputParams const & runparams) + OutputParams const & runparams_in) { - BufferStorage::iterator it = bstore.begin(); - BufferStorage::iterator end = bstore.end(); - for (; it != end; ++it) { - if (!(*it)->isDepClean(masterTmpDir)) { - string writefile = addName(masterTmpDir, (*it)->latexName()); - (*it)->makeLaTeXFile(FileName(writefile), masterTmpDir, - runparams, false); - (*it)->markDepClean(masterTmpDir); + OutputParams runparams = runparams_in; + runparams.is_child = true; + for (Buffer * buf : bstore) { + if (!buf->isDepClean(masterTmpDir)) { + string writefile = addName(masterTmpDir, buf->latexName()); + buf->makeLaTeXFile(FileName(writefile), masterTmpDir, + runparams, Buffer::OnlyBody); + buf->markDepClean(masterTmpDir); } } } @@ -204,132 +233,141 @@ void BufferList::updateIncludedTeXfiles(string const & masterTmpDir, void BufferList::emergencyWriteAll() { - for_each(bstore.begin(), bstore.end(), - bind(&BufferList::emergencyWrite, this, _1)); + for (Buffer * buf : bstore) + buf->emergencyWrite(); } -void BufferList::emergencyWrite(Buffer * buf) +void BufferList::invalidateConverterCache() const { - // Use ::assert to avoid a loop, BOOST_ASSERT ends up calling ::assert - // compare with 0 to avoid pointer/interger comparison - // ::assert(buf != 0); - if (!buf) - return; - - // No need to save if the buffer has not changed. - if (buf->isClean()) - return; - - string const doc = buf->isUnnamed() - ? onlyFilename(buf->absFileName()) : buf->absFileName(); - - lyxerr << to_utf8( - bformat(_("LyX: Attempting to save document %1$s"), from_utf8(doc))) - << endl; - - // We try to save three places: - // 1) Same place as document. Unless it is an unnamed doc. - if (!buf->isUnnamed()) { - string s = buf->absFileName(); - s += ".emergency"; - lyxerr << " " << s << endl; - if (buf->writeFile(FileName(s))) { - buf->markClean(); - lyxerr << to_utf8(_(" Save seems successful. Phew.")) << endl; - return; - } else { - lyxerr << to_utf8(_(" Save failed! Trying...")) << endl; - } - } - - // 2) In HOME directory. - string s = addName(package().home_dir().absFilename(), buf->absFileName()); - s += ".emergency"; - lyxerr << ' ' << s << endl; - if (buf->writeFile(FileName(s))) { - buf->markClean(); - lyxerr << to_utf8(_(" Save seems successful. Phew.")) << endl; - return; - } - - lyxerr << to_utf8(_(" Save failed! Trying...")) << endl; - - // 3) In "/tmp" directory. - // MakeAbsPath to prepend the current - // drive letter on OS/2 - s = addName(package().temp_dir().absFilename(), buf->absFileName()); - s += ".emergency"; - lyxerr << ' ' << s << endl; - if (buf->writeFile(FileName(s))) { - buf->markClean(); - lyxerr << to_utf8(_(" Save seems successful. Phew.")) << endl; - return; - } - lyxerr << to_utf8(_(" Save failed! Bummer. Document is lost.")) << endl; + for (Buffer const * buf : bstore) + buf->params().invalidateConverterCache(); } -bool BufferList::exists(string const & s) const +bool BufferList::exists(FileName const & fname) const { - return find_if(bstore.begin(), bstore.end(), - bind(equal_to(), - bind(&Buffer::absFileName, _1), - s)) - != bstore.end(); + return getBuffer(fname) != nullptr; } bool BufferList::isLoaded(Buffer const * b) const { - BOOST_ASSERT(b); + if (!b) + return false; BufferStorage::const_iterator cit = find(bstore.begin(), bstore.end(), b); return cit != bstore.end(); } -Buffer * BufferList::getBuffer(string const & s) +bool BufferList::isInternal(Buffer const * b) const { - BufferStorage::iterator it = - find_if(bstore.begin(), bstore.end(), - bind(equal_to(), - bind(&Buffer::absFileName, _1), - s)); + if (!b) + return false; + BufferStorage::const_iterator cit = + find(binternal.begin(), binternal.end(), b); + return cit != binternal.end(); +} - return it != bstore.end() ? (*it) : 0; + +bool BufferList::isOthersChild(Buffer * parent, Buffer * child) const +{ + LASSERT(parent, return false); + LASSERT(child, return false); + LASSERT(parent->isChild(child), return false); + + // Does child document have a different parent? + Buffer const * parent_ = child->parent(); + if (parent_ && parent_ != parent) + return true; + + for(Buffer const * buf : bstore) + if (buf != parent && buf->isChild(child)) + return true; + return false; } -Buffer * BufferList::getBufferFromTmp(string const & s) +Buffer * BufferList::getBuffer(support::FileName const & fname, bool internal) const { - BufferStorage::iterator it = bstore.begin(); - BufferStorage::iterator end = bstore.end(); - for (; it < end; ++it) - if (prefixIs(s, (*it)->temppath())) - return *it; - return 0; + // 1) cheap test, using string comparison of file names + for (Buffer * b : bstore) + if (b->fileName() == fname) + return b; + // 2) possibly expensive test, using equivalence test of file names + for (Buffer * b : bstore) + if (equivalent(b->fileName(), fname)) + return b; + if (internal) { + // 1) cheap test, using string comparison of file names + for (Buffer * b : binternal) + if (b->fileName() == fname) + return b; + // 2) possibly expensive test, using equivalence test of file names + for (Buffer * b : binternal) + if (equivalent(b->fileName(), fname)) + return b; + } + return nullptr; } -void BufferList::setCurrentAuthor(docstring const & name, docstring const & email) +Buffer * BufferList::getBufferFromTmp(string const & path, bool realpath) { - BufferStorage::iterator it = bstore.begin(); - BufferStorage::iterator end = bstore.end(); - for (; it != end; ++it) - (*it)->params().authors().record(0, Author(name, email)); + for (Buffer * buf : bstore) { + string const temppath = realpath ? FileName(buf->temppath()).realPath() : buf->temppath(); + if (prefixIs(path, temppath)) { + // check whether the filename matches the master + string const master_name = buf->latexName(); + if (suffixIs(path, master_name)) + return buf; + // if not, try with the children + for (Buffer * child : buf->getDescendants()) { + string const mangled_child_name = DocFileName( + changeExtension(child->absFileName(), + ".tex")).mangledFileName(); + if (suffixIs(path, mangled_child_name)) + return child; + } + } + } + return nullptr; } -int BufferList::bufferNum(string const & name) const +void BufferList::recordCurrentAuthor(Author const & author) { - vector buffers = getFileNames(); - vector::const_iterator cit = - find(buffers.begin(), buffers.end(), name); + for (Buffer * buf : bstore) + buf->params().authors().recordCurrentAuthor(author); +} + + +void BufferList::updatePreviews() +{ + for (Buffer * buf : bstore) + buf->updatePreviews(); +} + + +int BufferList::bufferNum(FileName const & fname) const +{ + FileNameList const buffers(fileNames()); + FileNameList::const_iterator cit = + find(buffers.begin(), buffers.end(), fname); if (cit == buffers.end()) return 0; return int(cit - buffers.begin()); } +void BufferList::changed(bool update_metrics) const +{ + for (Buffer const * buf : bstore) + buf->changed(update_metrics); + for (Buffer const * buf : binternal) + buf->changed(update_metrics); +} + + } // namespace lyx