* 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.
*/
#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 <boost/bind.hpp>
+#include "support/lassert.h"
#include <algorithm>
-#include <functional>
-
-using boost::bind;
+#include <iterator>
+#include <memory>
using namespace std;
using namespace lyx::support;
{}
+BufferList::~BufferList()
+{
+ for (Buffer * buf : binternal)
+ delete buf;
+}
+
+
bool BufferList::empty() const
{
return bstore.empty();
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)
+{
+ Buffer * const buf = createNewBuffer(s);
+ if (buf) {
+ LYXERR(Debug::INFO, "Assigning to buffer " << bstore.size());
+ bstore.push_back(buf);
}
+ return buf;
}
-Buffer * BufferList::newBuffer(string const & s, bool const ronly)
+Buffer * BufferList::createNewBuffer(string const & s)
{
- auto_ptr<Buffer> tmpbuf;
- try { tmpbuf.reset(new Buffer(s, ronly));
+ unique_ptr<Buffer> tmpbuf;
+ try {
+ tmpbuf = lyx::make_unique<Buffer>(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();
}
}
-vector<string> const BufferList::getFileNames() const
+FileNameList BufferList::fileNames() const
{
- vector<string> 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;
}
Buffer * BufferList::first()
{
if (bstore.empty())
- return 0;
+ return nullptr;
return bstore.front();
}
Buffer * BufferList::last()
{
if (bstore.empty())
- return 0;
+ return nullptr;
return bstore.back();
}
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);
}
}
}
void BufferList::emergencyWriteAll()
{
- for_each(bstore.begin(), bstore.end(),
- bind(&BufferList::emergencyWrite, this, _1));
+ for (Buffer * buf : bstore)
+ buf->emergencyWrite();
}
-docstring 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 file open!");
-
- // No need to save if the buffer has not changed.
- if (buf->isClean())
- return docstring();
-
- string const doc = buf->isUnnamed()
- ? onlyFilename(buf->absFileName()) : buf->absFileName();
-
- docstring user_message = bformat(
- _("LyX: Attempting to save document %1$s\n"), from_utf8(doc));
-
- // 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();
- user_message += _(" Save seems successful. Phew.\n");
- return user_message;
- } else {
- user_message += _(" Save failed! Trying...\n");
- }
- }
-
- // 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();
- user_message += _(" Save seems successful. Phew.\n");
- return user_message;
- }
-
- user_message += _(" Save failed! Trying...\n");
-
- // 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();
- user_message += _(" Save seems successful. Phew.\n");
- return user_message;
- }
-
- user_message += _(" Save failed! Bummer. Document is lost.");
- return user_message;
+ 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<string>(),
- 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<string>(),
- 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<string> buffers = getFileNames();
- vector<string>::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