]> git.lyx.org Git - lyx.git/blobdiff - src/buffer_funcs.cpp
Fix #10778 (issue with CJK and language nesting)
[lyx.git] / src / buffer_funcs.cpp
index 2167bd137c5baa4c613a7f1fa31bbcc1edba8739..beab239fc31215dda29f21e2c7f4e9df803dd5e2 100644 (file)
@@ -26,9 +26,9 @@
 #include "LaTeX.h"
 #include "Layout.h"
 #include "LyX.h"
+#include "LyXVC.h"
 #include "TextClass.h"
 #include "Paragraph.h"
-#include "paragraph_funcs.h"
 #include "ParagraphList.h"
 #include "ParagraphParameters.h"
 #include "ParIterator.h"
@@ -47,6 +47,7 @@
 #include "support/filetools.h"
 #include "support/gettext.h"
 #include "support/lstrings.h"
+#include "support/mutex.h"
 #include "support/textutils.h"
 
 using namespace std;
@@ -62,39 +63,59 @@ Buffer * checkAndLoadLyXFile(FileName const & filename, bool const acceptDirty)
        // File already open?
        Buffer * checkBuffer = theBufferList().getBuffer(filename);
        if (checkBuffer) {
-               // sometimes (when setting the master buffer from a child)
+               // Sometimes (when setting the master buffer from a child)
                // we accept a dirty buffer right away (otherwise we'd get
-               // an infinite loop (bug 5514)
-               if (checkBuffer->isClean() || acceptDirty)
+               // an infinite loop (bug 5514).
+               // We also accept a dirty buffer when the document has not
+               // yet been saved to disk.
+               if (checkBuffer->isClean() || acceptDirty || !filename.exists())
                        return checkBuffer;
-               docstring const file = makeDisplayPath(filename.absFilename(), 20);
-               docstring text = bformat(_(
+               docstring const file = makeDisplayPath(filename.absFileName(), 20);
+               docstring const text = bformat(_(
                                "The document %1$s is already loaded and has unsaved changes.\n"
                                "Do you want to abandon your changes and reload the version on disk?"), file);
-               if (Alert::prompt(_("Reload saved document?"),
-                               text, 0, 1,  _("&Reload"), _("&Keep Changes")))
-                       return checkBuffer;
-
-               // FIXME: should be LFUN_REVERT
-               theBufferList().release(checkBuffer);
-               // Load it again.
-               return checkAndLoadLyXFile(filename);
+               int res = Alert::prompt(_("Reload saved document?"),
+                                       text, 2, 2,  _("Yes, &Reload"), _("No, &Keep Changes"), _("&Cancel"));
+               switch (res) {
+                       case 0: {
+                               // reload the document
+                               if (checkBuffer->reload() != Buffer::ReadSuccess)
+                                       return 0;
+                               return checkBuffer;
+                       }
+                       case 1:
+                               // keep changes
+                               return checkBuffer;
+                       case 2:
+                               // cancel
+                               return 0;
+               }
        }
 
-       if (filename.exists()) {
-               if (!filename.isReadableFile()) {
-                       docstring text = bformat(_("The file %1$s exists but is not "
-                               "readable by the current user."),
-                               from_utf8(filename.absFilename()));
-                       Alert::error(_("File not readable!"), text);
-                       return 0;
+       bool const exists = filename.exists();
+       bool const tryVC = exists ? false : LyXVC::fileInVC(filename);
+       if (exists || tryVC) {
+               if (exists) {
+                       if (!filename.isReadableFile()) {
+                               docstring text = bformat(_("The file %1$s exists but is not "
+                                       "readable by the current user."),
+                                       from_utf8(filename.absFileName()));
+                               Alert::error(_("File not readable!"), text);
+                               return 0;
+                       }
+                       if (filename.extension() == "lyx" && filename.isFileEmpty()) {
+                               // Makes it possible to open an empty (0 bytes) .lyx file
+                               return newFile(filename.absFileName(), "", true);
+                       }
                }
-               Buffer * b = theBufferList().newBuffer(filename.absFilename());
+               Buffer * b = theBufferList().newBuffer(filename.absFileName());
                if (!b) {
                        // Buffer creation is not possible.
                        return 0;
                }
-               if (!b->loadLyXFile(filename)) {
+               if (b->loadLyXFile() != Buffer::ReadSuccess) {
+                       // do not save an emergency file when releasing the buffer
+                       b->markClean();
                        theBufferList().release(b);
                        return 0;
                }
@@ -103,10 +124,10 @@ Buffer * checkAndLoadLyXFile(FileName const & filename, bool const acceptDirty)
 
        docstring text = bformat(_("The document %1$s does not yet "
                "exist.\n\nDo you want to create a new document?"),
-               from_utf8(filename.absFilename()));
+               from_utf8(filename.absFileName()));
        if (!Alert::prompt(_("Create new document?"),
                        text, 0, 1, _("&Create"), _("Cancel")))
-               return newFile(filename.absFilename(), string(), true);
+               return newFile(filename.absFileName(), string(), true);
 
        return 0;
 }
@@ -114,7 +135,7 @@ Buffer * checkAndLoadLyXFile(FileName const & filename, bool const acceptDirty)
 
 // FIXME newFile() should probably be a member method of Application...
 Buffer * newFile(string const & filename, string const & templatename,
-                bool const isNamed)
+                bool is_named)
 {
        // get a free buffer
        Buffer * b = theBufferList().newBuffer(filename);
@@ -130,8 +151,8 @@ Buffer * newFile(string const & filename, string const & templatename,
                tname = makeAbsPath(templatename);
 
        if (!tname.empty()) {
-               if (!b->readFile(tname)) {
-                       docstring const file = makeDisplayPath(tname.absFilename(), 50);
+               if (b->loadThisLyXFile(tname) != Buffer::ReadSuccess) {
+                       docstring const file = makeDisplayPath(tname.absFileName(), 50);
                        docstring const text  = bformat(
                                _("The specified document template\n%1$s\ncould not be read."),
                                file);
@@ -141,10 +162,12 @@ Buffer * newFile(string const & filename, string const & templatename,
                }
        }
 
-       if (!isNamed) {
+       if (is_named)
+               // in this case, the user chose the filename, so we
+               // assume that she really does want this file.
+               b->markDirty();
+       else
                b->setUnnamed();
-               b->setFileName(filename);
-       }
 
        b->setReadonly(false);
        b->setFullyLoaded(true);
@@ -153,72 +176,45 @@ Buffer * newFile(string const & filename, string const & templatename,
 }
 
 
-Buffer * newUnnamedFile(string const & templatename, FileName const & path)
+Buffer * newUnnamedFile(FileName const & path, string const & prefix,
+                                               string const & templatename)
 {
-       static int newfile_number;
-
-       FileName filename(path, 
-               "newfile" + convert<string>(++newfile_number) + ".lyx");
-       while (theBufferList().exists(filename)
-               || filename.isReadableFile()) {
-               ++newfile_number;
-               filename.set(path,
-                       "newfile" +     convert<string>(newfile_number) + ".lyx");
-       }
-       return newFile(filename.absFilename(), templatename, false);
-}
+       static map<string, int> file_number;
+       static Mutex mutex;
 
+       Mutex::Locker locker(&mutex);
+       FileName filename;
 
-int countWords(DocIterator const & from, DocIterator const & to)
-{
-       int count = 0;
-       bool inword = false;
-       for (DocIterator dit = from ; dit != to ; dit.forwardPos()) {
-               // FIXME: use Paragraph::isWordSeparator
-               // Copied and adapted from isWordSeparator() in Paragraph
-               if (dit.inTexted()
-                   && dit.pos() != dit.lastpos()
-                   && !dit.paragraph().isWordSeparator(dit.pos())
-                   && !dit.paragraph().isDeleted(dit.pos())) {
-                       if (!inword) {
-                               ++count;
-                               inword = true;
-                       }
-               } else if (inword)
-                       inword = false;
+       do {
+               filename.set(path, 
+                       prefix + convert<string>(++file_number[prefix]) + ".lyx");
        }
-
-       return count;
+       while (theBufferList().exists(filename) || filename.isReadableFile());
+               
+       return newFile(filename.absFileName(), templatename, false);
 }
 
 
-int countChars(DocIterator const & from, DocIterator const & to, bool with_blanks)
+Buffer * loadIfNeeded(FileName const & fname)
 {
-       int chars = 0;
-       int blanks = 0;
-       for (DocIterator dit = from ; dit != to ; dit.forwardPos()) {
-
-               if (!dit.inTexted()) continue;
-               Paragraph const & par = dit.paragraph();
-               pos_type const pos = dit.pos();
-
-               if (pos != dit.lastpos() && !par.isDeleted(pos)) {
-                       if (Inset const * ins = par.getInset(pos)) {
-                               if (ins->isLetter())
-                                       ++chars;
-                               else if (with_blanks && ins->isSpace())
-                                       ++blanks;
-                       } else {
-                               char_type const c = par.getChar(pos);
-                               if (isPrintableNonspace(c))
-                                       ++chars;
-                               else if (isSpace(c) && with_blanks)
-                                       ++blanks;
-                       }
+       Buffer * buffer = theBufferList().getBuffer(fname);
+       if (!buffer) {
+               if (!fname.exists() && !LyXVC::fileInVC(fname))
+                       return 0;
+
+               buffer = theBufferList().newBuffer(fname.absFileName());
+               if (!buffer)
+                       // Buffer creation is not possible.
+                       return 0;
+
+               if (buffer->loadLyXFile() != Buffer::ReadSuccess) {
+                       //close the buffer we just opened
+                       theBufferList().release(buffer);
+                       return 0;
                }
        }
-
-       return chars + blanks;
+       return buffer;
 }
 
+
 } // namespace lyx