]> git.lyx.org Git - lyx.git/blobdiff - src/graphics/GraphicsConverter.cpp
de.po: fix doubled accelerator keys
[lyx.git] / src / graphics / GraphicsConverter.cpp
index 76572a00ef25e32fb6d52cc3fd2afa9a16965b58..6242ed91cb8e4b14e436cffbe4665ff5afc0a559 100644 (file)
 #include "GraphicsConverter.h"
 
 #include "Converter.h"
-#include "debug.h"
 #include "Format.h"
 
-#include "support/filetools.h"
-#include "support/ForkedCallQueue.h"
+#include "support/lassert.h"
 #include "support/convert.h"
+#include "support/debug.h"
+#include "support/FileName.h"
+#include "support/filetools.h"
+#include "support/ForkedCalls.h"
 #include "support/lstrings.h"
-#include "support/lyxlib.h"
 #include "support/os.h"
 
 #include <boost/bind.hpp>
 #include <sstream>
 #include <fstream>
 
-namespace support = lyx::support;
-
-using support::addExtension;
-using support::changeExtension;
-using support::FileName;
-using support::Forkedcall;
-using support::ForkedCallQueue;
-using support::getExtension;
-using support::libScriptSearch;
-using support::onlyPath;
-using support::onlyFilename;
-using support::quoteName;
-using support::quote_python;
-using support::subst;
-using support::tempName;
-using support::unlink;
-
-using std::endl;
-using std::ostream;
-using std::ostringstream;
-using std::string;
-
+using namespace std;
+using namespace lyx::support;
 
 namespace lyx {
+
 namespace graphics {
 
 class Converter::Impl : public boost::signals::trackable {
@@ -102,9 +84,10 @@ Converter::Converter(FileName const & from_file, string const & to_file_base,
 {}
 
 
-// Empty d-tor out-of-line to keep boost::scoped_ptr happy.
 Converter::~Converter()
-{}
+{
+       delete pimpl_;
+}
 
 
 void Converter::startConversion() const
@@ -137,11 +120,11 @@ Converter::Impl::Impl(FileName const & from_file, string const & to_file_base,
                      string const & from_format, string const & to_format)
        : valid_process_(false), finished_(false)
 {
-       LYXERR(Debug::GRAPHICS) << "Converter c-tor:\n"
+       LYXERR(Debug::GRAPHICS, "Converter c-tor:\n"
                << "\tfrom_file:      " << from_file
                << "\n\tto_file_base: " << to_file_base
                << "\n\tfrom_format:  " << from_format
-               << "\n\tto_format:    " << to_format << endl;
+               << "\n\tto_format:    " << to_format);
 
        // The converted image is to be stored in this file (we do not
        // use ChangeExtension because this is a basename which may
@@ -151,22 +134,22 @@ Converter::Impl::Impl(FileName const & from_file, string const & to_file_base,
        // The conversion commands are stored in a stringstream
        ostringstream script;
        build_script(from_file, to_file_base, from_format, to_format, script);
-       LYXERR(Debug::GRAPHICS) << "\tConversion script:"
-               << "\n--------------------------------------\n"
+       LYXERR(Debug::GRAPHICS, "\tConversion script:"
+                  "\n--------------------------------------\n"
                << script.str()
-               << "\n--------------------------------------\n";
+               << "\n--------------------------------------\n");
 
        // Output the script to file.
        static int counter = 0;
        script_file_ = FileName(onlyPath(to_file_base) + "lyxconvert" +
                convert<string>(counter++) + ".py");
 
-       std::ofstream fs(script_file_.toFilesystemEncoding().c_str());
+       ofstream fs(script_file_.toFilesystemEncoding().c_str());
        if (!fs.good()) {
                lyxerr << "Unable to write the conversion script to \""
                       << script_file_ << '\n'
                       << "Please check your directory permissions."
-                      << std::endl;
+                      << endl;
                return;
        }
 
@@ -177,7 +160,7 @@ Converter::Impl::Impl(FileName const & from_file, string const & to_file_base,
        // We create a dummy command for ease of understanding of the
        // list of forked processes.
        // Note: 'python ' is absolutely essential, or execvp will fail.
-       script_command_ = support::os::python() + ' ' +
+       script_command_ = os::python() + ' ' +
                quoteName(script_file_.toFilesystemEncoding()) + ' ' +
                quoteName(onlyFilename(from_file.toFilesystemEncoding())) + ' ' +
                quoteName(to_format);
@@ -193,13 +176,12 @@ void Converter::Impl::startConversion()
                return;
        }
 
-       Forkedcall::SignalTypePtr
-               ptr = ForkedCallQueue::get().add(script_command_);
-
+       ForkedCall::SignalTypePtr ptr =
+               ForkedCallQueue::add(script_command_);
        ptr->connect(boost::bind(&Impl::converted, this, _1, _2));
-
 }
 
+
 void Converter::Impl::converted(pid_t /* pid */, int retval)
 {
        if (finished_)
@@ -208,10 +190,10 @@ void Converter::Impl::converted(pid_t /* pid */, int retval)
 
        finished_ = true;
        // Clean-up behind ourselves
-       unlink(script_file_);
+       script_file_.removeFile();
 
        if (retval > 0) {
-               unlink(to_file_);
+               to_file_.removeFile();
                to_file_.erase();
                finishedConversion(false);
        } else {
@@ -226,8 +208,8 @@ static string const move_file(string const & from_file, string const & to_file)
                return string();
 
        ostringstream command;
-       command << "fromfile = " << from_file << "\n"
-               << "tofile = "   << to_file << "\n\n"
+       command << "fromfile = utf8ToDefaultEncoding(" << from_file << ")\n"
+               << "tofile = utf8ToDefaultEncoding("   << to_file << ")\n\n"
                << "try:\n"
                << "  os.rename(fromfile, tofile)\n"
                << "except:\n"
@@ -276,8 +258,8 @@ static void build_script(FileName const & from_file,
                  string const & to_format,
                  ostream & script)
 {
-       BOOST_ASSERT(from_format != to_format);
-       LYXERR(Debug::GRAPHICS) << "build_script ... ";
+       LASSERT(from_format != to_format, /**/);
+       LYXERR(Debug::GRAPHICS, "build_script ... ");
        typedef Converters::EdgePath EdgePath;
 
        script << "#!/usr/bin/env python\n"
@@ -312,8 +294,7 @@ static void build_script(FileName const & from_file,
        // Remember to remove the temp file because we only want the name...
        static int counter = 0;
        string const tmp = "gconvert" + convert<string>(counter++);
-       FileName const to_base(tempName(FileName(), tmp));
-       unlink(to_base);
+       FileName const to_base = FileName::tempName(tmp);
 
        // Create a copy of the file in case the original name contains
        // problematic characters like ' or ". We can work around that problem
@@ -323,7 +304,8 @@ static void build_script(FileName const & from_file,
        script << "infile = utf8ToDefaultEncoding("
                        << quoteName(from_file.absFilename(), quote_python)
                        << ")\n"
-                 "outfile = " << quoteName(outfile, quote_python) << "\n"
+                 "outfile = utf8ToDefaultEncoding("
+                       << quoteName(outfile, quote_python) << ")\n"
                  "shutil.copy(infile, outfile)\n";
 
        // Some converters (e.g. lilypond) can only output files to the
@@ -331,18 +313,19 @@ static void build_script(FileName const & from_file,
        // This has the added benefit that all other files that may be
        // generated by the converter are deleted when LyX closes and do not
        // clutter the real working directory.
-       script << "os.chdir(" << quoteName(onlyPath(outfile)) << ")\n";
+       script << "os.chdir(utf8ToDefaultEncoding("
+              << quoteName(onlyPath(outfile)) << "))\n";
 
        if (edgepath.empty()) {
                // Either from_format is unknown or we don't have a
                // converter path from from_format to to_format, so we use
                // the default converter.
                script << "infile = outfile\n"
-                      << "outfile = " << quoteName(to_file, quote_python)
-                      << '\n';
+                      << "outfile = utf8ToDefaultEncoding("
+                      << quoteName(to_file, quote_python) << ")\n";
 
                ostringstream os;
-               os << support::os::python() << ' '
+               os << os::python() << ' '
                   << libScriptSearch("$$s/scripts/convertDefault.py",
                                      quote_python) << ' ';
                if (!from_format.empty())
@@ -354,18 +337,18 @@ static void build_script(FileName const & from_file,
                   << to_format << ":' + '\"' + outfile + '\"' + '";
                string const command = os.str();
 
-               LYXERR(Debug::GRAPHICS)
-                       << "\tNo converter defined! I use convertDefault.py\n\t"
-                       << command << endl;
+               LYXERR(Debug::GRAPHICS,
+                       "\tNo converter defined! I use convertDefault.py\n\t"
+                       << command);
 
                build_conversion_command(command, script);
        }
 
        // The conversion commands may contain these tokens that need to be
        // changed to infile, infile_base, outfile respectively.
-       string const token_from("$$i");
-       string const token_base("$$b");
-       string const token_to("$$o");
+       string const token_from = "$$i";
+       string const token_base = "$$b";
+       string const token_to   = "$$o";
 
        EdgePath::const_iterator it  = edgepath.begin();
        EdgePath::const_iterator end = edgepath.end();
@@ -379,9 +362,12 @@ static void build_script(FileName const & from_file,
                outfile = addExtension(to_base.absFilename(), conv.To->extension());
 
                // Store these names in the python script
-               script << "infile = "      << quoteName(infile, quote_python) << "\n"
-                         "infile_base = " << quoteName(infile_base, quote_python) << "\n"
-                         "outfile = "     << quoteName(outfile, quote_python) << '\n';
+               script << "infile = utf8ToDefaultEncoding("
+                               << quoteName(infile, quote_python) << ")\n"
+                         "infile_base = utf8ToDefaultEncoding("
+                               << quoteName(infile_base, quote_python) << ")\n"
+                         "outfile = utf8ToDefaultEncoding("
+                               << quoteName(outfile, quote_python) << ")\n";
 
                // See comment about extra " quotes above (although that
                // applies only for the first loop run here).
@@ -396,9 +382,8 @@ static void build_script(FileName const & from_file,
 
        // Move the final outfile to to_file
        script << move_file("outfile", quoteName(to_file, quote_python));
-       LYXERR(Debug::GRAPHICS) << "ready!" << endl;
+       LYXERR(Debug::GRAPHICS, "ready!");
 }
 
 } // namespace graphics
-
 } // namespace lyx