]> git.lyx.org Git - lyx.git/blobdiff - src/lyx_main.C
do not create invalid .lyx files when importing \i{}, \j{}, \l{} or \L{}.
[lyx.git] / src / lyx_main.C
index a029f52e4162998e0b88feb265c52b5ff6d77cb1..ca36f2c081c1e808128c08db1b7a632e9178e1d7 100644 (file)
@@ -64,6 +64,7 @@
 
 #include <iostream>
 #include <csignal>
+#include <vector>
 
 
 namespace lyx {
@@ -163,6 +164,9 @@ struct LyX::Singletons
 
        ///
        IconvProcessor iconv;
+
+       /// Files to load at start.
+       vector<FileName> files_to_load_;
 };
 
 ///
@@ -331,16 +335,16 @@ int LyX::exec(int & argc, char * argv[])
        support::init_package(argv[0], cl_system_support, cl_user_support,
                                   support::top_build_dir_is_one_level_up);
 
-       vector<FileName> files;
-       
        if (!use_gui) {
                // FIXME: create a ConsoleApplication
-               int exit_status = loadFiles(argc, argv, files);
+               int exit_status = init(argc, argv);
                if (exit_status) {
                        prepareExit();
                        return exit_status;
                }
 
+               loadFiles();
+
                if (batch_command.empty() || pimpl_->buffer_list_.empty()) {
                        prepareExit();
                        return EXIT_SUCCESS;
@@ -368,20 +372,6 @@ int LyX::exec(int & argc, char * argv[])
 
        initGuiFont();
 
-       // Parse and remove all known arguments in the LyX singleton
-       // Give an error for all remaining ones.
-       int exit_status = loadFiles(argc, argv, files);
-       if (exit_status) {
-               // Kill the application object before exiting.
-               pimpl_->application_.reset();
-               use_gui = false;
-               prepareExit();
-               return exit_status;
-       }
-
-       restoreGuiSession(files);
-       // Start the real execution loop.
-
        // FIXME
        /* Create a CoreApplication class that will provide the main event loop
        * and the socket callback registering. With Qt4, only QtCore
@@ -393,6 +383,18 @@ int LyX::exec(int & argc, char * argv[])
        pimpl_->lyx_socket_.reset(new LyXServerSocket(&pimpl_->lyxfunc_, 
                support::os::internal_path(package().temp_dir() + "/lyxsocket")));
 
+       // Parse and remove all known arguments in the LyX singleton
+       // Give an error for all remaining ones.
+       int exit_status = init(argc, argv);
+       if (exit_status) {
+               // Kill the application object before exiting.
+               pimpl_->application_.reset();
+               use_gui = false;
+               prepareExit();
+               return exit_status;
+       }
+
+       // Start the real execution loop.
        exit_status = pimpl_->application_->exec();
        
        prepareExit();
@@ -424,7 +426,7 @@ void LyX::prepareExit()
                return;
        }
 
-       if (!destroyDir(package().temp_dir())) {
+       if (!destroyDir(FileName(package().temp_dir()))) {
                docstring const msg =
                        bformat(_("Unable to remove the temporary directory %1$s"),
                        from_utf8(package().temp_dir()));
@@ -439,8 +441,8 @@ void LyX::prepareExit()
                pimpl_->lyx_socket_.reset();
        }
 
-       // Kill the application object before exiting. This avoid crash
-       // on exit on Linux.
+       // Kill the application object before exiting. This avoids crashes
+       // when exiting on Linux.
        if (pimpl_->application_)
                pimpl_->application_.reset();
 }
@@ -456,8 +458,7 @@ void LyX::earlyExit(int status)
 }
 
 
-int LyX::loadFiles(int & argc, char * argv[],
-       vector<FileName> & files)
+int LyX::init(int & argc, char * argv[])
 {
        // check for any spurious extra arguments
        // other than documents
@@ -478,51 +479,48 @@ int LyX::loadFiles(int & argc, char * argv[],
                return EXIT_FAILURE;
 
        for (int argi = argc - 1; argi >= 1; --argi) {
-               // check for any remaining extra arguments other than
-               // document file names. These will be passed out to the
-               // frontend.
-               if (argv[argi][0] == '-')
-                       continue;
                // get absolute path of file and add ".lyx" to
                // the filename if necessary
-               files.push_back(fileSearch(string(), os::internal_path(argv[argi]), "lyx"));
+               pimpl_->files_to_load_.push_back(fileSearch(string(), os::internal_path(argv[argi]), "lyx"));
        }
 
        if (first_start)
-               files.push_back(i18nLibFileSearch("examples", "splash.lyx"));
+               pimpl_->files_to_load_.push_back(i18nLibFileSearch("examples", "splash.lyx"));
+
+       return EXIT_SUCCESS;
+}
 
-       Buffer * last_loaded = 0;
 
-       vector<FileName>::const_iterator it = files.begin();
-       vector<FileName>::const_iterator end = files.end();
+void LyX::loadFiles()
+{
+       vector<FileName>::const_iterator it = pimpl_->files_to_load_.begin();
+       vector<FileName>::const_iterator end = pimpl_->files_to_load_.end();
 
        for (; it != end; ++it) {
-               if (it->empty()) {
-                       Buffer * const b = newFile(it->absFilename(), string(), true);
-                       if (b)
-                               last_loaded = b;
-               } else {
-                       Buffer * buf = pimpl_->buffer_list_.newBuffer(it->absFilename(), false);
-                       if (loadLyXFile(buf, *it)) {
-                               last_loaded = buf;
-                               ErrorList const & el = buf->errorList("Parse");
-                               if (!el.empty())
-                                       for_each(el.begin(), el.end(),
-                                       boost::bind(&LyX::printError, this, _1));
-                       }
-                       else
-                               pimpl_->buffer_list_.release(buf);
+               if (it->empty())
+                       continue;
+
+               Buffer * const b = newFile(it->absFilename(), string(), true);
+               Buffer * buf = pimpl_->buffer_list_.newBuffer(it->absFilename(), false);
+               if (loadLyXFile(buf, *it)) {
+                       ErrorList const & el = buf->errorList("Parse");
+                       if (!el.empty())
+                               for_each(el.begin(), el.end(),
+                               boost::bind(&LyX::printError, this, _1));
                }
+               else
+                       pimpl_->buffer_list_.release(buf);
        }
-
-       files.clear(); // the files are already loaded
-
-       return EXIT_SUCCESS;
 }
 
 
 void LyX::execBatchCommands()
 {
+       // The advantage of doing this here is that the event loop
+       // is already started. So any need for interaction will be
+       // aknowledged.
+       restoreGuiSession();
+
        // Execute batch commands if available
        if (batch_command.empty())
                return;
@@ -534,23 +532,32 @@ void LyX::execBatchCommands()
 }
 
 
-void LyX::restoreGuiSession(vector<FileName> const & files)
+void LyX::restoreGuiSession()
 {
        LyXView * view = newLyXView();
 
-       // load files
-       for_each(files.begin(), files.end(),
-               bind(&LyXView::loadLyXFile, view, _1, true));
-
-       // if a file is specified, I assume that user wants to edit *that* file
-       if (files.empty() && lyxrc.load_session) {
-               vector<FileName> const & lastopened = pimpl_->session_->lastOpened().getfiles();
-               // do not add to the lastfile list since these files are restored from
-               // last seesion, and should be already there (regular files), or should
-               // not be added at all (help files).
-               for_each(lastopened.begin(), lastopened.end(),
-                       bind(&LyXView::loadLyXFile, view, _1, false));
+       // if some files were specified at command-line we assume that the
+       // user wants to edit *these* files and not to restore the session.
+       if (!pimpl_->files_to_load_.empty()) {
+               for_each(pimpl_->files_to_load_.begin(),
+                       pimpl_->files_to_load_.end(),
+                       bind(&LyXView::loadLyXFile, view, _1, true));
+               // clear this list to save a few bytes of RAM
+               pimpl_->files_to_load_.clear();
+               pimpl_->session_->lastOpened().clear();
+               return;
        }
+
+       if (!lyxrc.load_session)
+               return;
+
+       vector<FileName> const & lastopened = pimpl_->session_->lastOpened().getfiles();
+       // do not add to the lastfile list since these files are restored from
+       // last session, and should be already there (regular files), or should
+       // not be added at all (help files).
+       for_each(lastopened.begin(), lastopened.end(),
+               bind(&LyXView::loadLyXFile, view, _1, false));
+
        // clear this list to save a few bytes of RAM
        pimpl_->session_->lastOpened().clear();
 }
@@ -829,8 +836,9 @@ bool LyX::init()
        if (!lyxrc.path_prefix.empty())
                prependEnvPath("PATH", lyxrc.path_prefix);
 
-       if (fs::exists(lyxrc.document_path) &&
-           fs::is_directory(lyxrc.document_path))
+       FileName const document_path(lyxrc.document_path);
+       if (fs::exists(document_path.toFilesystemEncoding()) &&
+           fs::is_directory(document_path.toFilesystemEncoding()))
                package().document_dir() = lyxrc.document_path;
 
        package().temp_dir() = createLyXTmpDir(FileName(lyxrc.tempdir_path)).absFilename();
@@ -963,11 +971,19 @@ namespace {
 // return true if file does not exist or is older than configure.py.
 bool needsUpdate(string const & file)
 {
-       static string const configure_script =
-               addName(package().system_support(), "configure.py");
-       string const absfile =
-               addName(package().user_support(), file);
+       // We cannot initialize configure_script directly because the package
+       // is not initialized yet when  static objects are constructed.
+       static string configure_script;
+       static bool firstrun = true;
+       if (firstrun) {
+               configure_script = FileName(addName(
+                               package().system_support(),
+                               "configure.py")).toFilesystemEncoding();
+               firstrun = false;
+       }
 
+       string const absfile = FileName(addName(
+               package().user_support(), file)).toFilesystemEncoding();
        return (! fs::exists(absfile))
                || (fs::last_write_time(configure_script)
                    > fs::last_write_time(absfile));
@@ -979,8 +995,9 @@ bool needsUpdate(string const & file)
 bool LyX::queryUserLyXDir(bool explicit_userdir)
 {
        // Does user directory exist?
-       if (fs::exists(package().user_support()) &&
-           fs::is_directory(package().user_support())) {
+       string const user_support =
+               FileName(package().user_support()).toFilesystemEncoding();
+       if (fs::exists(user_support) && fs::is_directory(user_support)) {
                first_start = false;
 
                return needsUpdate("lyxrc.defaults")