]> git.lyx.org Git - features.git/blobdiff - src/Buffer.cpp
* docstream: factorize out some code and introduce odocfstream::reset()
[features.git] / src / Buffer.cpp
index 15bd4e468db3701962da9f4aa5f00db63ab9c5ed..7ab95e8f0ea53360609a82439d2add74e86ac26e 100644 (file)
 #include "frontends/alert.h"
 #include "frontends/Delegates.h"
 #include "frontends/WorkAreaManager.h"
-#include "frontends/FileDialog.h"
 
 #include "graphics/Previews.h"
 
 #include "support/convert.h"
 #include "support/debug.h"
 #include "support/FileFilterList.h"
+#include "support/FileNameList.h"
 #include "support/filetools.h"
 #include "support/ForkedCalls.h"
 #include "support/gettext.h"
@@ -91,6 +91,7 @@
 #include "support/Path.h"
 #include "support/textutils.h"
 #include "support/types.h"
+#include "support/FileZipListDir.h"
 
 #if !defined (HAVE_FORK)
 # define fork() -1
@@ -129,6 +130,7 @@ using support::changeExtension;
 using support::cmd_ret;
 using support::createBufferTmpDir;
 using support::FileName;
+using support::FileNameList;
 using support::libFileSearch;
 using support::latex_path;
 using support::ltrim;
@@ -152,7 +154,7 @@ namespace os = support::os;
 
 namespace {
 
-int const LYX_FORMAT = 303; // Uwe: Serbocroatian
+int const LYX_FORMAT = 304; // JSpitzm: framed.sty goes InsetBox
 
 } // namespace anon
 
@@ -163,11 +165,20 @@ class Buffer::Impl
 {
 public:
        Impl(Buffer & parent, FileName const & file, bool readonly);
+
+       ~Impl()
+       {
+               if (wa_) {
+                       wa_->closeAll();
+                       delete wa_;
+               }
+       }
        
        BufferParams params;
        LyXVC lyxvc;
        string temppath;
-       TexRow texrow;
+       mutable TexRow texrow;
+       Buffer const * parent_buffer;
 
        /// need to regenerate .tex?
        DepClean dep_clean;
@@ -197,7 +208,7 @@ public:
        InsetText inset;
 
        ///
-       TocBackend toc_backend;
+       mutable TocBackend toc_backend;
 
        /// macro table
        typedef std::map<unsigned int, MacroData, std::greater<int> > PositionToMacroMap;
@@ -220,14 +231,18 @@ public:
 
        ///
        Undo undo_;
+
+       /// A cache for the bibfiles (including bibfiles of loaded child
+       /// documents), needed for appropriate update of natbib labels.
+       mutable FileNameList bibfilesCache_;
 };
 
 
 Buffer::Impl::Impl(Buffer & parent, FileName const & file, bool readonly_)
-       : lyx_clean(true), bak_clean(true), unnamed(false), read_only(readonly_),
-         filename(file), file_fully_loaded(false), inset(params),
-         toc_backend(&parent), embedded_files(&parent), timestamp_(0),
-         checksum_(0), wa_(0), undo_(parent)
+       : parent_buffer(0), lyx_clean(true), bak_clean(true), unnamed(false),
+         read_only(readonly_), filename(file), file_fully_loaded(false),
+         inset(params), toc_backend(&parent), embedded_files(&parent),
+         timestamp_(0), checksum_(0), wa_(0), undo_(parent)
 {
        inset.setAutoBreakRows(true);
        lyxvc.setBuffer(&parent);
@@ -243,7 +258,7 @@ Buffer::Impl::Impl(Buffer & parent, FileName const & file, bool readonly_)
 
 
 Buffer::Buffer(string const & file, bool readonly)
-       : pimpl_(new Impl(*this, FileName(file), readonly)), gui_(0)
+       : d(new Impl(*this, FileName(file), readonly)), gui_(0)
 {
        LYXERR(Debug::INFO, "Buffer::Buffer()");
 }
@@ -255,13 +270,18 @@ Buffer::~Buffer()
        // here the buffer should take care that it is
        // saved properly, before it goes into the void.
 
-       Buffer * master = masterBuffer();
+       // GuiView already destroyed
+       gui_ = 0;
+
+       Buffer const * master = masterBuffer();
        if (master != this && use_gui)
                // We are closing buf which was a child document so we
                // must update the labels and section numbering of its master
                // Buffer.
                updateLabels(*master);
 
+       resetChildDocuments(false);
+
        if (!temppath().empty() && !FileName(temppath()).destroyDirectory()) {
                Alert::warning(_("Could not remove temporary directory"),
                        bformat(_("Could not remove the temporary directory %1$s"),
@@ -271,49 +291,45 @@ Buffer::~Buffer()
        // Remove any previewed LaTeX snippets associated with this buffer.
        graphics::Previews::get().removeLoader(*this);
 
-       if (pimpl_->wa_) {
-               pimpl_->wa_->closeAll();
-               delete pimpl_->wa_;
-       }
-       delete pimpl_;
+       delete d;
 }
 
 
 void Buffer::changed() const
 {
-       if (pimpl_->wa_)
-               pimpl_->wa_->redrawAll();
+       if (d->wa_)
+               d->wa_->redrawAll();
 }
 
 
 frontend::WorkAreaManager & Buffer::workAreaManager() const
 {
-       BOOST_ASSERT(pimpl_->wa_);
-       return *pimpl_->wa_;
+       BOOST_ASSERT(d->wa_);
+       return *d->wa_;
 }
 
 
 Text & Buffer::text() const
 {
-       return const_cast<Text &>(pimpl_->inset.text_);
+       return const_cast<Text &>(d->inset.text_);
 }
 
 
 Inset & Buffer::inset() const
 {
-       return const_cast<InsetText &>(pimpl_->inset);
+       return const_cast<InsetText &>(d->inset);
 }
 
 
 BufferParams & Buffer::params()
 {
-       return pimpl_->params;
+       return d->params;
 }
 
 
 BufferParams const & Buffer::params() const
 {
-       return pimpl_->params;
+       return d->params;
 }
 
 
@@ -331,67 +347,55 @@ ParagraphList const & Buffer::paragraphs() const
 
 LyXVC & Buffer::lyxvc()
 {
-       return pimpl_->lyxvc;
+       return d->lyxvc;
 }
 
 
 LyXVC const & Buffer::lyxvc() const
 {
-       return pimpl_->lyxvc;
+       return d->lyxvc;
 }
 
 
 string const & Buffer::temppath() const
 {
-       return pimpl_->temppath;
-}
-
-
-TexRow & Buffer::texrow()
-{
-       return pimpl_->texrow;
+       return d->temppath;
 }
 
 
 TexRow const & Buffer::texrow() const
 {
-       return pimpl_->texrow;
+       return d->texrow;
 }
 
 
-TocBackend & Buffer::tocBackend()
+TocBackend & Buffer::tocBackend() const
 {
-       return pimpl_->toc_backend;
-}
-
-
-TocBackend const & Buffer::tocBackend() const
-{
-       return pimpl_->toc_backend;
+       return d->toc_backend;
 }
 
 
 EmbeddedFiles & Buffer::embeddedFiles()
 {
-       return pimpl_->embedded_files;
+       return d->embedded_files;
 }
 
 
 EmbeddedFiles const & Buffer::embeddedFiles() const
 {
-       return pimpl_->embedded_files;
+       return d->embedded_files;
 }
 
 
 Undo & Buffer::undo()
 {
-       return pimpl_->undo_;
+       return d->undo_;
 }
 
 
 string Buffer::latexName(bool const no_path) const
 {
-       FileName latex_name = makeLatexName(pimpl_->filename);
+       FileName latex_name = makeLatexName(d->filename);
        return no_path ? latex_name.onlyFileName()
                : latex_name.absFilename();
 }
@@ -435,8 +439,8 @@ string Buffer::logName(LogType * type) const
 
 void Buffer::setReadonly(bool const flag)
 {
-       if (pimpl_->read_only != flag) {
-               pimpl_->read_only = flag;
+       if (d->read_only != flag) {
+               d->read_only = flag;
                setReadOnly(flag);
        }
 }
@@ -444,8 +448,8 @@ void Buffer::setReadonly(bool const flag)
 
 void Buffer::setFileName(string const & newfile)
 {
-       pimpl_->filename = makeAbsPath(newfile);
-       setReadonly(pimpl_->filename.isReadOnly());
+       d->filename = makeAbsPath(newfile);
+       setReadonly(d->filename.isReadOnly());
        updateTitles();
 }
 
@@ -479,7 +483,7 @@ int Buffer::readHeader(Lexer & lex)
                params().temp_bullet(i) = ITEMIZE_DEFAULTS[i];
        }
 
-       ErrorList & errorList = pimpl_->errorLists["Parse"];
+       ErrorList & errorList = d->errorLists["Parse"];
 
        while (lex.isOK()) {
                lex.next();
@@ -500,7 +504,7 @@ int Buffer::readHeader(Lexer & lex)
                LYXERR(Debug::PARSER, "Handling document header token: `"
                                      << token << '\'');
 
-               string unknown = params().readToken(lex, token, pimpl_->filename.onlyPath());
+               string unknown = params().readToken(lex, token, d->filename.onlyPath());
                if (!unknown.empty()) {
                        if (unknown[0] != '\\' && token == "\\textclass") {
                                Alert::warning(_("Unknown document class"),
@@ -532,7 +536,7 @@ int Buffer::readHeader(Lexer & lex)
 // Returns false if "\end_document" is not read (Asger)
 bool Buffer::readDocument(Lexer & lex)
 {
-       ErrorList & errorList = pimpl_->errorLists["Parse"];
+       ErrorList & errorList = d->errorLists["Parse"];
        errorList.clear();
 
        lex.next();
@@ -703,13 +707,13 @@ bool Buffer::readFile(FileName const & filename)
 
 bool Buffer::isFullyLoaded() const
 {
-       return pimpl_->file_fully_loaded;
+       return d->file_fully_loaded;
 }
 
 
 void Buffer::setFullyLoaded(bool value)
 {
-       pimpl_->file_fully_loaded = value;
+       d->file_fully_loaded = value;
 }
 
 
@@ -757,7 +761,7 @@ Buffer::ReadStatus Buffer::readFile(Lexer & lex, FileName const & filename,
        // save timestamp and checksum of the original disk file, making sure
        // to not overwrite them with those of the file created in the tempdir
        // when it has to be converted to the current format.
-       if (!pimpl_->checksum_) {
+       if (!d->checksum_) {
                // Save the timestamp and checksum of disk file. If filename is an
                // emergency file, save the timestamp and checksum of the original lyx file
                // because isExternallyModified will check for this file. (BUG4193)
@@ -826,7 +830,7 @@ Buffer::ReadStatus Buffer::readFile(Lexer & lex, FileName const & filename,
                                       from_utf8(filename.absFilename())));
        }
 
-       pimpl_->file_fully_loaded = true;
+       d->file_fully_loaded = true;
        return success;
 }
 
@@ -837,7 +841,7 @@ bool Buffer::save() const
        // We don't need autosaves in the immediate future. (Asger)
        resetAutosaveTimers();
 
-       string const encodedFilename = pimpl_->filename.toFilesystemEncoding();
+       string const encodedFilename = d->filename.toFilesystemEncoding();
 
        FileName backupName;
        bool madeBackup = false;
@@ -873,15 +877,13 @@ bool Buffer::save() const
                        return false;
        }
 
-       if (writeFile(pimpl_->filename)) {
+       if (writeFile(d->filename)) {
                markClean();
-               removeAutosaveFile(absFileName());
-               saveCheckSum(pimpl_->filename);
                return true;
        } else {
                // Saving failed, so backup is not backup
                if (madeBackup)
-                       rename(backupName, pimpl_->filename);
+                       rename(backupName, d->filename);
                return false;
        }
 }
@@ -889,7 +891,7 @@ bool Buffer::save() const
 
 bool Buffer::writeFile(FileName const & fname) const
 {
-       if (pimpl_->read_only && fname == pimpl_->filename)
+       if (d->read_only && fname == d->filename)
                return false;
 
        bool retval = false;
@@ -900,27 +902,40 @@ bool Buffer::writeFile(FileName const & fname) const
                content = FileName(addName(temppath(), "content.lyx"));
        else
                content = fname;
-       
+
+       docstring const str = bformat(_("Saving document %1$s..."),
+               makeDisplayPath(content.absFilename()));
+       message(str);
+
        if (params().compressed) {
                gz::ogzstream ofs(content.toFilesystemEncoding().c_str(), ios::out|ios::trunc);
-               if (!ofs)
-                       return false;
-
-               retval = write(ofs);
+               retval = ofs && write(ofs);
        } else {
                ofstream ofs(content.toFilesystemEncoding().c_str(), ios::out|ios::trunc);
-               if (!ofs)
-                       return false;
+               retval = ofs && write(ofs);
+       }
 
-               retval = write(ofs);
+       if (!retval) {
+               message(str + _(" could not write file!."));
+               return false;
        }
 
-       if (retval && params().embedded) {
-               // write file.lyx and all the embedded files to the zip file fname
-               // if embedding is enabled
-               return pimpl_->embedded_files.writeFile(fname);
+       removeAutosaveFile(d->filename.absFilename());
+       saveCheckSum(d->filename);
+       message(str + _(" done."));
+
+       if (!params().embedded)
+               return true;
+
+       message(str + _(" writing embedded files!."));
+       // if embedding is enabled, write file.lyx and all the embedded files
+       // to the zip file fname.
+       if (!d->embedded_files.writeFile(fname)) {
+               message(str + _(" could not write embedded files!."));
+               return false;
        }
-       return retval;
+       message(str + _(" error while writing embedded files."));
+       return true;
 }
 
 
@@ -989,12 +1004,20 @@ bool Buffer::write(ostream & ofs) const
 bool Buffer::makeLaTeXFile(FileName const & fname,
                           string const & original_path,
                           OutputParams const & runparams,
-                          bool output_preamble, bool output_body)
+                          bool output_preamble, bool output_body) const
 {
        string const encoding = runparams.encoding->iconvName();
        LYXERR(Debug::LATEX, "makeLaTeXFile encoding: " << encoding << "...");
 
-       odocfstream ofs(encoding);
+       odocfstream ofs;
+       try { ofs.reset(encoding); }
+       catch (iconv_codecvt_facet_exception & e) {
+               lyxerr << "Caught iconv exception: " << e.what() << endl;
+               Alert::error(_("Iconv software exception Detected"), bformat(_("Please "
+                       "verify that the support software for your encoding (%1$s) is "
+                       "properly installed"), from_ascii(encoding)));
+               return false;
+       }
        if (!openFileWrite(ofs, fname))
                return false;
 
@@ -1002,7 +1025,7 @@ bool Buffer::makeLaTeXFile(FileName const & fname,
 
        bool failed_export = false;
        try {
-               texrow().reset();
+               d->texrow.reset();
                writeLaTeXSource(ofs, original_path,
                      runparams, output_preamble, output_body);
        }
@@ -1040,7 +1063,7 @@ bool Buffer::makeLaTeXFile(FileName const & fname,
 void Buffer::writeLaTeXSource(odocstream & os,
                           string const & original_path,
                           OutputParams const & runparams_in,
-                          bool const output_preamble, bool const output_body)
+                          bool const output_preamble, bool const output_body) const
 {
        OutputParams runparams = runparams_in;
 
@@ -1057,8 +1080,8 @@ void Buffer::writeLaTeXSource(odocstream & os,
                        "For more info, see http://www.lyx.org/.\n"
                        "%% Do not edit unless you really know what "
                        "you are doing.\n";
-               texrow().newline();
-               texrow().newline();
+               d->texrow.newline();
+               d->texrow.newline();
        }
        LYXERR(Debug::INFO, "lyx document header finished");
        // There are a few differences between nice LaTeX and usual files:
@@ -1076,7 +1099,7 @@ void Buffer::writeLaTeXSource(odocstream & os,
                        // code for usual, NOT nice-latex-file
                        os << "\\batchmode\n"; // changed
                        // from \nonstopmode
-                       texrow().newline();
+                       d->texrow.newline();
                }
                if (!original_path.empty()) {
                        // FIXME UNICODE
@@ -1086,23 +1109,23 @@ void Buffer::writeLaTeXSource(odocstream & os,
                           << "\\def\\input@path{{"
                           << inputpath << "/}}\n"
                           << "\\makeatother\n";
-                       texrow().newline();
-                       texrow().newline();
-                       texrow().newline();
+                       d->texrow.newline();
+                       d->texrow.newline();
+                       d->texrow.newline();
                }
 
                // Write the preamble
-               runparams.use_babel = params().writeLaTeX(os, features, texrow());
+               runparams.use_babel = params().writeLaTeX(os, features, d->texrow);
 
                if (!output_body)
                        return;
 
                // make the body.
                os << "\\begin{document}\n";
-               texrow().newline();
+               d->texrow.newline();
        } // output_preamble
 
-       texrow().start(paragraphs().begin()->id(), 0);
+       d->texrow.start(paragraphs().begin()->id(), 0);
        
        LYXERR(Debug::INFO, "preamble finished, now the body.");
 
@@ -1113,7 +1136,7 @@ void Buffer::writeLaTeXSource(odocstream & os,
                                           "$$lang",
                                           params().language->babel()))
                   << '\n';
-               texrow().newline();
+               d->texrow.newline();
        }
 
        Encoding const & encoding = params().encoding();
@@ -1123,37 +1146,37 @@ void Buffer::writeLaTeXSource(odocstream & os,
                // the preamble if it is handled by CJK.sty.
                os << "\\begin{CJK}{" << from_ascii(encoding.latexName())
                   << "}{}\n";
-               texrow().newline();
+               d->texrow.newline();
        }
 
        // if we are doing a real file with body, even if this is the
        // child of some other buffer, let's cut the link here.
        // This happens for example if only a child document is printed.
-       string save_parentname;
+       Buffer const * save_parent = 0;
        if (output_preamble) {
-               save_parentname = params().parentname;
-               params().parentname.erase();
+               save_parent = d->parent_buffer;
+               d->parent_buffer = 0;
        }
 
        loadChildDocuments();
 
        // the real stuff
-       latexParagraphs(*this, paragraphs(), os, texrow(), runparams);
+       latexParagraphs(*this, paragraphs(), os, d->texrow, runparams);
 
        // Restore the parenthood if needed
        if (output_preamble)
-               params().parentname = save_parentname;
+               d->parent_buffer = save_parent;
 
        // add this just in case after all the paragraphs
        os << endl;
-       texrow().newline();
+       d->texrow.newline();
 
        if (encoding.package() == Encoding::CJK) {
                // Close the open CJK environment.
                // latexParagraphs will have opened one even if the last text
                // was not CJK.
                os << "\\end{CJK}\n";
-               texrow().newline();
+               d->texrow.newline();
        }
 
        if (!lyxrc.language_auto_end &&
@@ -1162,12 +1185,12 @@ void Buffer::writeLaTeXSource(odocstream & os,
                                           "$$lang",
                                           params().language->babel()))
                   << '\n';
-               texrow().newline();
+               d->texrow.newline();
        }
 
        if (output_preamble) {
                os << "\\end{document}\n";
-               texrow().newline();
+               d->texrow.newline();
                LYXERR(Debug::LATEX, "makeLaTeXFile...done");
        } else {
                LYXERR(Debug::LATEX, "LaTeXFile for inclusion made.");
@@ -1175,10 +1198,10 @@ void Buffer::writeLaTeXSource(odocstream & os,
        runparams_in.encoding = runparams.encoding;
 
        // Just to be sure. (Asger)
-       texrow().newline();
+       d->texrow.newline();
 
        LYXERR(Debug::INFO, "Finished making LaTeX file.");
-       LYXERR(Debug::INFO, "Row count was " << texrow().rows() - 1 << '.');
+       LYXERR(Debug::INFO, "Row count was " << d->texrow.rows() - 1 << '.');
 }
 
 
@@ -1202,7 +1225,7 @@ bool Buffer::isDocBook() const
 
 void Buffer::makeDocBookFile(FileName const & fname,
                              OutputParams const & runparams,
-                             bool const body_only)
+                             bool const body_only) const
 {
        LYXERR(Debug::LATEX, "makeDocBookFile...");
 
@@ -1221,12 +1244,12 @@ void Buffer::makeDocBookFile(FileName const & fname,
 
 void Buffer::writeDocBookSource(odocstream & os, string const & fname,
                             OutputParams const & runparams,
-                            bool const only_body)
+                            bool const only_body) const
 {
        LaTeXFeatures features(*this, params(), runparams);
        validate(features);
 
-       texrow().reset();
+       d->texrow.reset();
 
        TextClass const & tclass = params().getTextClass();
        string const top_element = tclass.latexname();
@@ -1322,7 +1345,7 @@ int Buffer::runChktex()
                Alert::error(_("chktex failure"),
                             _("Could not run chktex successfully."));
        } else if (res > 0) {
-               ErrorList & errlist = pimpl_->errorLists["ChkTeX"];
+               ErrorList & errlist = d->errorLists["ChkTeX"];
                errlist.clear();
                bufferErrors(terr, errlist);
        }
@@ -1430,33 +1453,33 @@ void Buffer::getLabelList(vector<docstring> & list) const
 }
 
 
-void Buffer::updateBibfilesCache()
+void Buffer::updateBibfilesCache() const
 {
        // if this is a child document and the parent is already loaded
        // update the parent's cache instead
-       Buffer * tmp = masterBuffer();
+       Buffer const * tmp = masterBuffer();
        BOOST_ASSERT(tmp);
        if (tmp != this) {
                tmp->updateBibfilesCache();
                return;
        }
 
-       bibfilesCache_.clear();
+       d->bibfilesCache_.clear();
        for (InsetIterator it = inset_iterator_begin(inset()); it; ++it) {
                if (it->lyxCode() == BIBTEX_CODE) {
                        InsetBibtex const & inset =
                                static_cast<InsetBibtex const &>(*it);
-                       vector<FileName> const bibfiles = inset.getFiles(*this);
-                       bibfilesCache_.insert(bibfilesCache_.end(),
+                       FileNameList const bibfiles = inset.getFiles(*this);
+                       d->bibfilesCache_.insert(d->bibfilesCache_.end(),
                                bibfiles.begin(),
                                bibfiles.end());
                } else if (it->lyxCode() == INCLUDE_CODE) {
                        InsetInclude & inset =
                                static_cast<InsetInclude &>(*it);
                        inset.updateBibfilesCache(*this);
-                       vector<FileName> const & bibfiles =
+                       FileNameList const & bibfiles =
                                        inset.getBibfilesCache(*this);
-                       bibfilesCache_.insert(bibfilesCache_.end(),
+                       d->bibfilesCache_.insert(d->bibfilesCache_.end(),
                                bibfiles.begin(),
                                bibfiles.end());
                }
@@ -1464,7 +1487,7 @@ void Buffer::updateBibfilesCache()
 }
 
 
-vector<FileName> const & Buffer::getBibfilesCache() const
+FileNameList const & Buffer::getBibfilesCache() const
 {
        // if this is a child document and the parent is already loaded
        // use the parent's cache instead
@@ -1474,17 +1497,17 @@ vector<FileName> const & Buffer::getBibfilesCache() const
                return tmp->getBibfilesCache();
 
        // We update the cache when first used instead of at loading time.
-       if (bibfilesCache_.empty())
+       if (d->bibfilesCache_.empty())
                const_cast<Buffer *>(this)->updateBibfilesCache();
 
-       return bibfilesCache_;
+       return d->bibfilesCache_;
 }
 
 
 bool Buffer::isDepClean(string const & name) const
 {
-       DepClean::const_iterator const it = pimpl_->dep_clean.find(name);
-       if (it == pimpl_->dep_clean.end())
+       DepClean::const_iterator const it = d->dep_clean.find(name);
+       if (it == d->dep_clean.end())
                return true;
        return it->second;
 }
@@ -1492,7 +1515,7 @@ bool Buffer::isDepClean(string const & name) const
 
 void Buffer::markDepClean(string const & name)
 {
-       pimpl_->dep_clean[name] = true;
+       d->dep_clean[name] = true;
 }
 
 
@@ -1607,80 +1630,80 @@ docstring const Buffer::B_(string const & l10n) const
 
 bool Buffer::isClean() const
 {
-       return pimpl_->lyx_clean;
+       return d->lyx_clean;
 }
 
 
 bool Buffer::isBakClean() const
 {
-       return pimpl_->bak_clean;
+       return d->bak_clean;
 }
 
 
 bool Buffer::isExternallyModified(CheckMethod method) const
 {
-       BOOST_ASSERT(pimpl_->filename.exists());
+       BOOST_ASSERT(d->filename.exists());
        // if method == timestamp, check timestamp before checksum
        return (method == checksum_method 
-               || pimpl_->timestamp_ != pimpl_->filename.lastModified())
-               && pimpl_->checksum_ != pimpl_->filename.checksum();
+               || d->timestamp_ != d->filename.lastModified())
+               && d->checksum_ != d->filename.checksum();
 }
 
 
 void Buffer::saveCheckSum(FileName const & file) const
 {
        if (file.exists()) {
-               pimpl_->timestamp_ = file.lastModified();
-               pimpl_->checksum_ = file.checksum();
+               d->timestamp_ = file.lastModified();
+               d->checksum_ = file.checksum();
        } else {
                // in the case of save to a new file.
-               pimpl_->timestamp_ = 0;
-               pimpl_->checksum_ = 0;
+               d->timestamp_ = 0;
+               d->checksum_ = 0;
        }
 }
 
 
 void Buffer::markClean() const
 {
-       if (!pimpl_->lyx_clean) {
-               pimpl_->lyx_clean = true;
+       if (!d->lyx_clean) {
+               d->lyx_clean = true;
                updateTitles();
        }
        // if the .lyx file has been saved, we don't need an
        // autosave
-       pimpl_->bak_clean = true;
+       d->bak_clean = true;
 }
 
 
 void Buffer::markBakClean() const
 {
-       pimpl_->bak_clean = true;
+       d->bak_clean = true;
 }
 
 
 void Buffer::setUnnamed(bool flag)
 {
-       pimpl_->unnamed = flag;
+       d->unnamed = flag;
 }
 
 
 bool Buffer::isUnnamed() const
 {
-       return pimpl_->unnamed;
+       return d->unnamed;
 }
 
 
 // FIXME: this function should be moved to buffer_pimpl.C
 void Buffer::markDirty()
 {
-       if (pimpl_->lyx_clean) {
-               pimpl_->lyx_clean = false;
+       if (d->lyx_clean) {
+               d->lyx_clean = false;
                updateTitles();
        }
-       pimpl_->bak_clean = false;
+       d->bak_clean = false;
 
-       DepClean::iterator it = pimpl_->dep_clean.begin();
-       DepClean::const_iterator const end = pimpl_->dep_clean.end();
+       DepClean::iterator it = d->dep_clean.begin();
+       DepClean::const_iterator const end = d->dep_clean.end();
 
        for (; it != end; ++it)
                it->second = false;
@@ -1689,79 +1712,59 @@ void Buffer::markDirty()
 
 FileName Buffer::fileName() const
 {
-       return pimpl_->filename;
+       return d->filename;
 }
 
 
 string Buffer::absFileName() const
 {
-       return pimpl_->filename.absFilename();
+       return d->filename.absFilename();
 }
 
 
 string Buffer::filePath() const
 {
-       return pimpl_->filename.onlyPath().absFilename();
+       return d->filename.onlyPath().absFilename();
 }
 
 
 bool Buffer::isReadonly() const
 {
-       return pimpl_->read_only;
+       return d->read_only;
 }
 
 
-void Buffer::setParentName(string const & name)
+void Buffer::setParent(Buffer const * buffer)
 {
-       if (name == pimpl_->filename.absFilename())
-               // Avoids recursive include.
-               params().parentname.clear();
-       else
-               params().parentname = name;
+       // Avoids recursive include.
+       d->parent_buffer = buffer == this ? 0 : buffer;
 }
 
 
-Buffer const * Buffer::masterBuffer() const
+Buffer const * Buffer::parent()
 {
-       if (!params().parentname.empty()
-               && theBufferList().exists(params().parentname)) {
-               Buffer const * buf = theBufferList().getBuffer(params().parentname);
-               //We need to check if the parent is us...
-               //FIXME RECURSIVE INCLUDE
-               //This is not sufficient, since recursive includes could be downstream.
-               if (buf && buf != this)
-                       return buf->masterBuffer();
-       }
-
-       return this;
+       return d->parent_buffer;
 }
 
 
-Buffer * Buffer::masterBuffer()
+Buffer const * Buffer::masterBuffer() const
 {
-       if (!params().parentname.empty()
-           && theBufferList().exists(params().parentname)) {
-               Buffer * buf = theBufferList().getBuffer(params().parentname);
-               //We need to check if the parent is us...
-               //FIXME RECURSIVE INCLUDE
-               //This is not sufficient, since recursive includes could be downstream.
-               if (buf && buf != this)
-                       return buf->masterBuffer();
-       }
-
-       return this;
+       if (!d->parent_buffer)
+               return this;
+       
+       return d->parent_buffer->masterBuffer();
 }
 
 
 bool Buffer::hasMacro(docstring const & name, Paragraph const & par) const
 {
        Impl::PositionToMacroMap::iterator it;
-       it = pimpl_->macros[name].upper_bound(par.macrocontextPosition());
-       if (it != pimpl_->macros[name].end())
+       it = d->macros[name].upper_bound(par.macrocontextPosition());
+       if (it != d->macros[name].end())
                return true;
 
        // If there is a master buffer, query that
-       const Buffer * master = masterBuffer();
+       Buffer const * master = masterBuffer();
        if (master && master != this)
                return master->hasMacro(name);
 
@@ -1771,11 +1774,11 @@ bool Buffer::hasMacro(docstring const & name, Paragraph const & par) const
 
 bool Buffer::hasMacro(docstring const & name) const
 {
-       if( !pimpl_->macros[name].empty() )
+       if( !d->macros[name].empty() )
                return true;
 
        // If there is a master buffer, query that
-       const Buffer * master = masterBuffer();
+       Buffer const * master = masterBuffer();
        if (master && master != this)
                return master->hasMacro(name);
 
@@ -1787,12 +1790,12 @@ MacroData const & Buffer::getMacro(docstring const & name,
        Paragraph const & par) const
 {
        Impl::PositionToMacroMap::iterator it;
-       it = pimpl_->macros[name].upper_bound(par.macrocontextPosition());
-       if( it != pimpl_->macros[name].end() )
+       it = d->macros[name].upper_bound(par.macrocontextPosition());
+       if( it != d->macros[name].end() )
                return it->second;
 
        // If there is a master buffer, query that
-       const Buffer * master = masterBuffer();
+       Buffer const * master = masterBuffer();
        if (master && master != this)
                return master->getMacro(name);
 
@@ -1803,12 +1806,12 @@ MacroData const & Buffer::getMacro(docstring const & name,
 MacroData const & Buffer::getMacro(docstring const & name) const
 {
        Impl::PositionToMacroMap::iterator it;
-       it = pimpl_->macros[name].begin();
-       if( it != pimpl_->macros[name].end() )
+       it = d->macros[name].begin();
+       if( it != d->macros[name].end() )
                return it->second;
 
        // If there is a master buffer, query that
-       const Buffer * master = masterBuffer();
+       Buffer const * master = masterBuffer();
        if (master && master != this)
                return master->getMacro(name);
 
@@ -1819,7 +1822,7 @@ MacroData const & Buffer::getMacro(docstring const & name) const
 void Buffer::updateMacros()
 {
        // start with empty table
-       pimpl_->macros = Impl::NameToPositionMacroMap();
+       d->macros = Impl::NameToPositionMacroMap();
 
        // Iterate over buffer
        ParagraphList & pars = text().paragraphs();
@@ -1850,7 +1853,7 @@ void Buffer::updateMacros()
                                macro.setRedefinition(hasMacro(macroTemplate.name()));
 
                                // register macro (possibly overwrite the previous one of this paragraph)
-                               pimpl_->macros[macroTemplate.name()][i] = macro;
+                               d->macros[macroTemplate.name()][i] = macro;
                        }
                }
        }
@@ -1905,11 +1908,11 @@ void Buffer::getSourceCode(odocstream & os, pit_type par_begin,
        // No side effect of file copying and image conversion
        runparams.dryrun = true;
 
-       texrow().reset();
+       d->texrow.reset();
        if (full_source) {
                os << "% " << _("Preview source code") << "\n\n";
-               texrow().newline();
-               texrow().newline();
+               d->texrow.newline();
+               d->texrow.newline();
                if (isLatex())
                        writeLaTeXSource(os, filePath(), runparams, true, true);
                else {
@@ -1928,11 +1931,11 @@ void Buffer::getSourceCode(odocstream & os, pit_type par_begin,
                                        convert<docstring>(par_begin),
                                        convert<docstring>(par_end - 1))
                           << "\n\n";
-               texrow().newline();
-               texrow().newline();
+               d->texrow.newline();
+               d->texrow.newline();
                // output paragraphs
                if (isLatex()) {
-                       latexParagraphs(*this, paragraphs(), os, texrow(), runparams);
+                       latexParagraphs(*this, paragraphs(), os, d->texrow, runparams);
                } else {
                        // DocBook
                        docbookParagraphs(paragraphs(), *this, os, runparams);
@@ -1941,23 +1944,17 @@ void Buffer::getSourceCode(odocstream & os, pit_type par_begin,
 }
 
 
-ErrorList const & Buffer::errorList(string const & type) const
+ErrorList & Buffer::errorList(string const & type) const
 {
-       static ErrorList const emptyErrorList;
-       std::map<string, ErrorList>::const_iterator I = pimpl_->errorLists.find(type);
-       if (I == pimpl_->errorLists.end())
+       static ErrorList emptyErrorList;
+       std::map<string, ErrorList>::iterator I = d->errorLists.find(type);
+       if (I == d->errorLists.end())
                return emptyErrorList;
 
        return I->second;
 }
 
 
-ErrorList & Buffer::errorList(string const & type)
-{
-       return pimpl_->errorLists[type];
-}
-
-
 void Buffer::structureChanged() const
 {
        if (gui_)
@@ -1988,15 +1985,15 @@ void Buffer::setBusy(bool on) const
 
 void Buffer::setReadOnly(bool on) const
 {
-       if (pimpl_->wa_)
-               pimpl_->wa_->setReadOnly(on);
+       if (d->wa_)
+               d->wa_->setReadOnly(on);
 }
 
 
 void Buffer::updateTitles() const
 {
-       if (pimpl_->wa_)
-               pimpl_->wa_->updateTitles();
+       if (d->wa_)
+               d->wa_->updateTitles();
 }
 
 
@@ -2109,7 +2106,7 @@ void Buffer::autoSave() const
        // create autosave filename
        string fname = filePath();
        fname += '#';
-       fname += onlyFilename(absFileName());
+       fname += d->filename.onlyFileName();
        fname += '#';
 
        AutoSaveBuffer autosave(*this, FileName(fname));
@@ -2120,112 +2117,19 @@ void Buffer::autoSave() const
 }
 
 
-/** Write a buffer to a new file name and rename the buffer
-    according to the new file name.
-
-    This function is e.g. used by menu callbacks and
-    LFUN_BUFFER_WRITE_AS.
-
-    If 'newname' is empty (the default), the user is asked via a
-    dialog for the buffer's new name and location.
-
-    If 'newname' is non-empty and has an absolute path, that is used.
-    Otherwise the base directory of the buffer is used as the base
-    for any relative path in 'newname'.
-*/
-
-bool Buffer::writeAs(string const & newname)
+void Buffer::resetChildDocuments(bool close_them) const
 {
-       string fname = absFileName();
-       string const oldname = fname;
-
-       if (newname.empty()) {  /// No argument? Ask user through dialog
-
-               // FIXME UNICODE
-               FileDialog dlg(_("Choose a filename to save document as"),
-                                  LFUN_BUFFER_WRITE_AS);
-               dlg.setButton1(_("Documents|#o#O"), from_utf8(lyxrc.document_path));
-               dlg.setButton2(_("Templates|#T#t"), from_utf8(lyxrc.template_path));
-
-               if (!support::isLyXFilename(fname))
-                       fname += ".lyx";
-
-               support::FileFilterList const filter(_("LyX Documents (*.lyx)"));
-
-               FileDialog::Result result =
-                       dlg.save(from_utf8(onlyPath(fname)),
-                                    filter,
-                                    from_utf8(onlyFilename(fname)));
-
-               if (result.first == FileDialog::Later)
-                       return false;
-
-               fname = to_utf8(result.second);
-
-               if (fname.empty())
-                       return false;
-
-               // Make sure the absolute filename ends with appropriate suffix
-               fname = makeAbsPath(fname).absFilename();
-               if (!support::isLyXFilename(fname))
-                       fname += ".lyx";
-
-       } else 
-               fname = makeAbsPath(newname, onlyPath(oldname)).absFilename();
-
-       if (FileName(fname).exists()) {
-               docstring const file = makeDisplayPath(fname, 30);
-               docstring text = bformat(_("The document %1$s already "
-                                          "exists.\n\nDo you want to "
-                                          "overwrite that document?"), 
-                                        file);
-               int const ret = Alert::prompt(_("Overwrite document?"),
-                       text, 0, 1, _("&Overwrite"), _("&Cancel"));
-
-               if (ret == 1)
-                       return false;
-       }
-
-       // Ok, change the name of the buffer
-       setFileName(fname);
-       markDirty();
-       bool unnamed = isUnnamed();
-       setUnnamed(false);
-       saveCheckSum(FileName(fname));
-
-       if (!menuWrite()) {
-               setFileName(oldname);
-               setUnnamed(unnamed);
-               saveCheckSum(FileName(oldname));
-               return false;
-       }
-
-       removeAutosaveFile(oldname);
-       return true;
-}
-
+       for (InsetIterator it = inset_iterator_begin(inset()); it; ++it) {
+               if (it->lyxCode() != INCLUDE_CODE)
+                       continue;
+               InsetCommand const & inset = static_cast<InsetCommand const &>(*it);
+               InsetCommandParams const & ip = inset.params();
 
-bool Buffer::menuWrite()
-{
-       if (save()) {
-               LyX::ref().session().lastFiles().add(FileName(absFileName()));
-               return true;
+               resetParentBuffer(this, ip, close_them);
        }
 
-       // FIXME: we don't tell the user *WHY* the save failed !!
-
-       docstring const file = makeDisplayPath(absFileName(), 30);
-
-       docstring text = bformat(_("The document %1$s could not be saved.\n\n"
-                                  "Do you want to rename the document and "
-                                  "try again?"), file);
-       int const ret = Alert::prompt(_("Rename and save?"),
-               text, 0, 1, _("&Rename"), _("&Cancel"));
-
-       if (ret != 0)
-               return false;
-
-       return writeAs();
+       if (use_gui && masterBuffer() == this)
+               updateLabels(*this);
 }
 
 
@@ -2261,7 +2165,7 @@ string Buffer::bufferFormat() const
 
 
 bool Buffer::doExport(string const & format, bool put_in_tempdir,
-       string & result_file)
+       string & result_file) const
 {
        string backend_format;
        OutputParams runparams(&params().encoding());
@@ -2379,14 +2283,14 @@ bool Buffer::doExport(string const & format, bool put_in_tempdir,
 }
 
 
-bool Buffer::doExport(string const & format, bool put_in_tempdir)
+bool Buffer::doExport(string const & format, bool put_in_tempdir) const
 {
        string result_file;
        return doExport(format, put_in_tempdir, result_file);
 }
 
 
-bool Buffer::preview(string const & format)
+bool Buffer::preview(string const & format) const
 {
        string result_file;
        if (!doExport(format, true, result_file))
@@ -2541,13 +2445,13 @@ void Buffer::bufferErrors(TeXErrors const & terr, ErrorList & errorList) const
                int id_start = -1;
                int pos_start = -1;
                int errorRow = cit->error_in_line;
-               bool found = texrow().getIdFromRow(errorRow, id_start,
+               bool found = d->texrow.getIdFromRow(errorRow, id_start,
                                                       pos_start);
                int id_end = -1;
                int pos_end = -1;
                do {
                        ++errorRow;
-                       found = texrow().getIdFromRow(errorRow, id_end, pos_end);
+                       found = d->texrow.getIdFromRow(errorRow, id_end, pos_end);
                } while (found && id_start == id_end && pos_start == pos_end);
 
                errorList.push_back(ErrorItem(cit->error_desc,