Simplify logic a bit. (This was meant to be part of a previous

commit.)

(cherry picked from commit 2104d41f71ff39f5c3f9f5a485707633b1bfb25b)

diff --git a/src/Buffer.cpp b/src/Buffer.cpp
index 22c993ca8b93401096459768007352e1e4599851..c20ce12b032f1752f01617fb1fc50ed63bfa50d5 100644 (file)
--- a/src/Buffer.cpp
+++ b/src/Buffer.cpp
@@ -4318,12 +4318,11 @@ Buffer::ExportStatus Buffer::preview(string const & format, bool includeall) con
 
        if (status != ExportSuccess)
                return status;
-       if (previewFile.exists()) {
-               if (!formats.view(*this, previewFile, format))
-                       return PreviewError;
-               else
-                       return PreviewSuccess;
-       } 
+
+       if (previewFile.exists())
+               return formats.view(*this, previewFile, format) ?
+                       PreviewSuccess : PreviewError;
+
        // Successful export but no output file?
        // Probably a bug in error detection.
        LATTEST(status != ExportSuccess);