commit fb2c558f674ba22dc66fda1cac937e903f26c7cd Author: Richard Heck <rgh...@lyx.org> Date: Sun Nov 15 12:24:37 2015 -0500
We need to show the error message in the case in which we copied the file due to a format change, as well. diff --git a/src/Buffer.cpp b/src/Buffer.cpp index b1807ed..eac9821 100644 --- a/src/Buffer.cpp +++ b/src/Buffer.cpp @@ -1333,7 +1333,7 @@ FileName Buffer::getBackupName() const { v = 1000; break; } - string newbackname = backname + "-" + convert<string>(v); + string const newbackname = backname + "-" + convert<string>(v); backup.set(addName(fpath, addExtension(newbackname, fext))); v++; } @@ -1401,7 +1401,8 @@ bool Buffer::save() const bool made_backup = true; FileName backupName; - if (lyxrc.make_backup || d->need_format_backup) { + bool const needBackup = lyxrc.make_backup || d->need_format_backup; + if (needBackup) { if (d->need_format_backup) backupName = getBackupName(); @@ -1458,8 +1459,8 @@ bool Buffer::save() const } // else we saved the file, but failed to move it to the right location. - if (lyxrc.make_backup && made_backup && !symlink) { - // the original file was moved to filename.lyx~, so it will look + if (needBackup && made_backup && !symlink) { + // the original file was moved to some new location, so it will look // to the user as if it was deleted. (see bug #9234.) we could try // to restore it, but that would basically mean trying to do again // what we just failed to do. better to leave things as they are.