// yet been saved to disk.
if (checkBuffer->isClean() || acceptDirty || !filename.exists())
return checkBuffer;
- docstring const file = makeDisplayPath(filename.absFilename(), 20);
+ 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"))) {
// reload the document
- if (!checkBuffer->reload())
+ if (checkBuffer->reload() != Buffer::ReadSuccess)
return 0;
}
return checkBuffer;
if (!filename.isReadableFile()) {
docstring text = bformat(_("The file %1$s exists but is not "
"readable by the current user."),
- from_utf8(filename.absFilename()));
+ from_utf8(filename.absFileName()));
Alert::error(_("File not readable!"), text);
return 0;
}
- 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);
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;
}
// 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);
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);
}
}
- 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);
}
while (theBufferList().exists(filename) || filename.isReadableFile());
- return newFile(filename.absFilename(), templatename, false);
+ return newFile(filename.absFileName(), templatename, false);
}
{
int count = 0;
bool inword = false;
- for (DocIterator dit = from ; dit != to ; ) {
+
+ for (DocIterator dit = from ; dit != to && !dit.empty(); ) {
if (!dit.inTexted()) {
dit.forwardPos();
continue;
Paragraph const & par = dit.paragraph();
pos_type const pos = dit.pos();
-
+
// Copied and adapted from isWordSeparator() in Paragraph
- if (pos != dit.lastpos() && !par.isDeleted(pos)) {
+ if (pos == dit.lastpos()) {
+ inword = false;
+ } else if (!par.isDeleted(pos)) {
Inset const * ins = par.getInset(pos);
if (ins && !ins->producesOutput()) {
//skip this inset
++dit.top().pos();
+ if (dit >= to)
+ break;
continue;
}
if (par.isWordSeparator(pos))
if (!fname.exists())
return 0;
- buffer = theBufferList().newBuffer(fname.absFilename());
+ buffer = theBufferList().newBuffer(fname.absFileName());
if (!buffer)
// Buffer creation is not possible.
return 0;
- if (!buffer->loadLyXFile(fname)) {
+ if (buffer->loadLyXFile() != Buffer::ReadSuccess) {
//close the buffer we just opened
theBufferList().release(buffer);
return 0;