X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2Flyx_main.C;h=caa2a2b6897730040b8fa97abcefda1667b6cf0a;hb=c4320d24cd2d29c2e77958b4a8fd44f2bd587ca7;hp=90d98fd03d9f965ac36e2a2ea111922c48f459ac;hpb=0dae88fca68636c487a8ab4102778868cf18ad53;p=lyx.git diff --git a/src/lyx_main.C b/src/lyx_main.C index 90d98fd03d..caa2a2b689 100644 --- a/src/lyx_main.C +++ b/src/lyx_main.C @@ -74,6 +74,7 @@ using support::bformat; using support::createDirectory; using support::createLyXTmpDir; using support::destroyDir; +using support::FileName; using support::fileSearch; using support::getEnv; using support::i18nLibFileSearch; @@ -98,8 +99,6 @@ using std::signal; using std::system; #endif -/// -frontend::Application * theApp = 0; /// are we using the GUI at all? /** @@ -115,6 +114,8 @@ namespace { string cl_system_support; string cl_user_support; +std::string geometryArg; + LyX * singleton_ = 0; void showFileError(string const & error) @@ -164,16 +165,18 @@ struct LyX::Singletons IconvProcessor iconv; }; +/// +frontend::Application * theApp() +{ + if (singleton_) + return &singleton_->application(); + else + return 0; +} + LyX::~LyX() { - // Static data are not treated in the same way at all on the Mac (and - // the LyX singleton has static methods). This is the reason why the - // exit command on the Mac bypasses our dispatch machinery altogether. - // On Linux and Windows we won't pass a second time through quit() - // because quitting will already be set to true. - if (!quitting) - quit(); } @@ -192,10 +195,11 @@ LyX const & LyX::cref() LyX::LyX() - : first_start(false), geometryOption_(false) + : first_start(false) { singleton_ = this; pimpl_.reset(new Singletons); + geometryArg.clear(); } @@ -327,7 +331,7 @@ 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 files; + vector files; if (!use_gui) { // FIXME: create a ConsoleApplication @@ -361,8 +365,6 @@ int LyX::exec(int & argc, char * argv[]) // Let the frontend parse and remove all arguments that it knows pimpl_->application_.reset(createApplication(argc, argv)); - // FIXME: this global pointer should probably go. - theApp = pimpl_->application_.get(); initGuiFont(); @@ -392,9 +394,9 @@ int LyX::exec(int & argc, char * argv[]) support::os::internal_path(package().temp_dir() + "/lyxsocket"))); exit_status = pimpl_->application_->exec(); - // Kill the application object before exiting. This avoid crash - // on exit on Linux. - pimpl_->application_.reset(); + + prepareExit(); + // Restore original font resources after Application is destroyed. support::restoreFontResources(); @@ -414,12 +416,33 @@ void LyX::prepareExit() // do any other cleanup procedures now lyxerr[Debug::INFO] << "Deleting tmp dir " << package().temp_dir() << endl; + // Prevent the deletion of /tmp if LyX was called with invalid + // arguments. Does not work on windows. + // FIXME: Fix the real bug instead. + if (package().temp_dir() == "/tmp") { + lyxerr << "Not deleting /tmp." << endl; + return; + } + if (!destroyDir(package().temp_dir())) { docstring const msg = bformat(_("Unable to remove the temporary directory %1$s"), from_utf8(package().temp_dir())); Alert::warning(_("Unable to remove temporary directory"), msg); } + + if (use_gui) { + if (pimpl_->session_) + pimpl_->session_->writeFile(); + pimpl_->session_.reset(); + pimpl_->lyx_server_.reset(); + pimpl_->lyx_socket_.reset(); + } + + // Kill the application object before exiting. This avoids crashes + // when exiting on Linux. + if (pimpl_->application_) + pimpl_->application_.reset(); } @@ -433,24 +456,8 @@ void LyX::earlyExit(int status) } -void LyX::quit() -{ - lyxerr[Debug::INFO] << "Running QuitLyX." << endl; - - prepareExit(); - - if (use_gui) { - pimpl_->session_->writeFile(); - pimpl_->lyx_server_.reset(); - pimpl_->lyx_socket_.reset(); - pimpl_->application_->exit(0); - theApp = 0; - } -} - - int LyX::loadFiles(int & argc, char * argv[], - vector & files) + vector & files) { // check for any spurious extra arguments // other than documents @@ -476,7 +483,9 @@ int LyX::loadFiles(int & argc, char * argv[], // frontend. if (argv[argi][0] == '-') continue; - files.push_back(os::internal_path(argv[argi])); + // get absolute path of file and add ".lyx" to + // the filename if necessary + files.push_back(fileSearch(string(), os::internal_path(argv[argi]), "lyx")); } if (first_start) @@ -484,20 +493,17 @@ int LyX::loadFiles(int & argc, char * argv[], Buffer * last_loaded = 0; - vector::const_iterator it = files.begin(); - vector::const_iterator end = files.end(); + vector::const_iterator it = files.begin(); + vector::const_iterator end = files.end(); for (; it != end; ++it) { - // get absolute path of file and add ".lyx" to - // the filename if necessary - string s = fileSearch(string(), *it, "lyx"); - if (s.empty()) { - Buffer * const b = newFile(*it, string(), true); + if (it->empty()) { + Buffer * const b = newFile(it->absFilename(), string(), true); if (b) last_loaded = b; } else { - Buffer * buf = pimpl_->buffer_list_.newBuffer(s, false); - if (loadLyXFile(buf, s)) { + 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()) @@ -528,7 +534,7 @@ void LyX::execBatchCommands() } -void LyX::restoreGuiSession(vector const & files) +void LyX::restoreGuiSession(vector const & files) { LyXView * view = newLyXView(); @@ -538,7 +544,7 @@ void LyX::restoreGuiSession(vector const & files) // if a file is specified, I assume that user wants to edit *that* file if (files.empty() && lyxrc.load_session) { - vector const & lastopened = pimpl_->session_->lastOpened().getfiles(); + vector 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). @@ -559,6 +565,8 @@ LyXView * LyX::newLyXView() // initial geometry unsigned int width = 690; unsigned int height = 510; + // default icon size, will be overwritten by stored session value + unsigned int iconSizeXY = 0; bool maximize = false; // first try lyxrc if (lyxrc.geometry_width != 0 && lyxrc.geometry_height != 0 ) { @@ -575,6 +583,9 @@ LyXView * LyX::newLyXView() height = convert(val); if (session().sessionInfo().load("WindowIsMaximized") == "yes") maximize = true; + val = session().sessionInfo().load("IconSizeXY"); + if (!val.empty()) + iconSizeXY = convert(val); } // if user wants to restore window position @@ -589,12 +600,14 @@ LyXView * LyX::newLyXView() posy = convert(val); } - if (geometryOption_) { + if (!geometryArg.empty()) + { width = 0; height = 0; } + // create the main window - LyXView * view = &pimpl_->application_->createView(width, height, posx, posy, maximize); + LyXView * view = &pimpl_->application_->createView(width, height, posx, posy, maximize, iconSizeXY, geometryArg); return view; } @@ -820,7 +833,7 @@ bool LyX::init() fs::is_directory(lyxrc.document_path)) package().document_dir() = lyxrc.document_path; - package().temp_dir() = createLyXTmpDir(lyxrc.tempdir_path); + package().temp_dir() = createLyXTmpDir(FileName(lyxrc.tempdir_path)).absFilename(); if (package().temp_dir().empty()) { Alert::error(_("Could not create temporary directory"), bformat(_("Could not create a temporary directory in\n" @@ -911,7 +924,8 @@ void LyX::emergencyCleanup() const pimpl_->buffer_list_.emergencyWriteAll(); if (use_gui) { - pimpl_->lyx_server_->emergencyCleanup(); + if (pimpl_->lyx_server_) + pimpl_->lyx_server_->emergencyCleanup(); pimpl_->lyx_server_.reset(); pimpl_->lyx_socket_.reset(); } @@ -1011,7 +1025,7 @@ bool LyX::readRcFile(string const & name) { lyxerr[Debug::INIT] << "About to read " << name << "... "; - string const lyxrc_path = libFileSearch(string(), name); + FileName const lyxrc_path = libFileSearch(string(), name); if (!lyxrc_path.empty()) { lyxerr[Debug::INIT] << "Found in " << lyxrc_path << endl; @@ -1060,7 +1074,7 @@ bool LyX::readUIFile(string const & name) lyxerr[Debug::INIT] << "About to read " << name << "..." << endl; - string const ui_path = libFileSearch("ui", name, "ui"); + FileName const ui_path = libFileSearch("ui", name, "ui"); if (ui_path.empty()) { lyxerr[Debug::INIT] << "Could not find " << name << endl; @@ -1118,7 +1132,7 @@ bool LyX::readLanguagesFile(string const & name) { lyxerr[Debug::INIT] << "About to read " << name << "..." << endl; - string const lang_path = libFileSearch(string(), name); + FileName const lang_path = libFileSearch(string(), name); if (lang_path.empty()) { showFileError(name); return false; @@ -1133,7 +1147,7 @@ bool LyX::readEncodingsFile(string const & name) { lyxerr[Debug::INIT] << "About to read " << name << "..." << endl; - string const enc_path = libFileSearch(string(), name); + FileName const enc_path = libFileSearch(string(), name); if (enc_path.empty()) { showFileError(name); return false; @@ -1259,6 +1273,19 @@ int parse_import(string const & type, string const & file) return 2; } +int parse_geometry(string const & arg1, string const &) +{ + geometryArg = arg1; +#if defined(_WIN32) || (defined(__CYGWIN__) && defined(X_DISPLAY_MISSING)) + // remove also the arg + return 1; +#else + // don't remove "-geometry" + return -1; +#endif +} + + } // namespace anon @@ -1279,15 +1306,12 @@ void LyX::easyParse(int & argc, char * argv[]) cmdmap["--export"] = parse_export; cmdmap["-i"] = parse_import; cmdmap["--import"] = parse_import; + cmdmap["-geometry"] = parse_geometry; for (int i = 1; i < argc; ++i) { std::map::const_iterator it = cmdmap.find(argv[i]); - // check for X11 -geometry option - if (support::compare(argv[i], "-geometry") == 0) - geometryOption_ = true; - // don't complain if not found - may be parsed later if (it == cmdmap.end()) continue; @@ -1299,10 +1323,12 @@ void LyX::easyParse(int & argc, char * argv[]) // Now, remove used arguments by shifting // the following ones remove places down. - argc -= remove; - for (int j = i; j < argc; ++j) - argv[j] = argv[j + remove]; - --i; + if (remove > 0) { + argc -= remove; + for (int j = i; j < argc; ++j) + argv[j] = argv[j + remove]; + --i; + } } batch_command = batch;