Squeeze warning.

[lyx.git] / src / Exporter.cpp

diff --git a/src/Exporter.cpp b/src/Exporter.cpp
index 85d5eb3d5551862f7791fc21897651da4124a5bc..a5a153aacdd7e0c069d5c3c279a2a1e5f0b8987c 100644 (file)
--- a/src/Exporter.cpp
+++ b/src/Exporter.cpp
@@ -36,18 +36,19 @@ namespace lyx {
 
 namespace Alert = frontend::Alert;
 
-/// ask the user what to do if a file already exists
+/// Ask the user what to do if a file already exists
 static int checkOverwrite(FileName const & filename)
 {
        if (!filename.exists())
-               return 0;
+               return 1;
+
        docstring text = bformat(_("The file %1$s already exists.\n\n"
-                                                        "Do you want to overwrite that file?"),
-                                               makeDisplayPath(filename.absFilename()));
+                                  "Do you want to overwrite that file?"),
+                                  makeDisplayPath(filename.absFileName()));
        return Alert::prompt(_("Overwrite file?"),
-                                        text, 0, 2,
-                                        _("&Overwrite"), _("Overwrite &all"),
-                                        _("&Cancel export"));
+                               text, 0, 3,
+                               _("&Keep file"), _("&Overwrite"),
+                               _("Overwrite &all"), _("&Cancel export"));
 }
 
 
@@ -69,15 +70,20 @@ CopyStatus copyFile(string const & format,
        // overwrite themselves. This check could be changed to
        // boost::filesystem::equivalent(sourceFile, destFile) if export to
        // other directories than the document directory is desired.
-       if (!prefixIs(onlyPath(sourceFile.absFilename()), package().temp_dir().absFilename()))
+       // Also don't overwrite files that already exist and are identical
+       // to the source files.
+       if (!prefixIs(onlyPath(sourceFile.absFileName()), package().temp_dir().absFileName())
+           || sourceFile.checksum() == destFile.checksum())
                return ret;
 
        if (!force) {
                switch(checkOverwrite(destFile)) {
                case 0:
+                       return SUCCESS;
+               case 1:
                        ret = SUCCESS;
                        break;
-               case 1:
+               case 2:
                        ret = FORCE;
                        break;
                default:
@@ -89,8 +95,8 @@ CopyStatus copyFile(string const & format,
        if (!mover.copy(sourceFile, destFile, latexFile))
                Alert::error(_("Couldn't copy file"),
                             bformat(_("Copying %1$s to %2$s failed."),
-                                    makeDisplayPath(sourceFile.absFilename()),
-                                    makeDisplayPath(destFile.absFilename())));
+                                    makeDisplayPath(sourceFile.absFileName()),
+                                    makeDisplayPath(destFile.absFileName())));
 
        return ret;
 }
@@ -125,7 +131,7 @@ void ExportData::addExternalFile(string const & format,
 void ExportData::addExternalFile(string const & format,
                                 FileName const & sourceName)
 {
-       addExternalFile(format, sourceName, onlyFilename(sourceName.absFilename()));
+       addExternalFile(format, sourceName, onlyFileName(sourceName.absFileName()));
 }