X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2FSession.cpp;h=310aa74c9b90760af8af76071c2cf5d471801c75;hb=ee0cc3cec1cf97e154e968d33dc580f37eb29e84;hp=36161e84ea410526cf7f8f86b056b08a17259fb3;hpb=f370da411306112060c52335dbd33bf28692ddd0;p=lyx.git diff --git a/src/Session.cpp b/src/Session.cpp index 36161e84ea..310aa74c9b 100644 --- a/src/Session.cpp +++ b/src/Session.cpp @@ -12,32 +12,18 @@ #include #include "Session.h" -#include "debug.h" -#include "support/Package.h" + +#include "support/debug.h" #include "support/filetools.h" +#include "support/Package.h" #include #include #include #include -using lyx::support::absolutePath; -using lyx::support::addName; -using lyx::support::FileName; -using lyx::support::package; - -using std::vector; -using std::getline; -using std::string; -using std::ifstream; -using std::ofstream; -using std::istream; -using std::ostream; -using std::endl; -using std::istringstream; -using std::copy; -using std::find; -using std::ostream_iterator; +using namespace std; +using namespace lyx::support; namespace { @@ -47,6 +33,8 @@ string const sec_lastopened = "[last opened files]"; string const sec_bookmarks = "[bookmarks]"; string const sec_session = "[session info]"; string const sec_toolbars = "[toolbars]"; +string const sec_lastcommands = "[last commands]"; +string const sec_authfiles = "[auth files]"; } // anon namespace @@ -69,7 +57,7 @@ void LastFilesSection::read(istream & is) if (c == '[') break; getline(is, tmp); - if (tmp == "" || tmp[0] == '#' || tmp[0] == ' ' || !absolutePath(tmp)) + if (tmp.empty() || tmp[0] == '#' || tmp[0] == ' ' || !FileName::isAbsolute(tmp)) continue; // read lastfiles @@ -78,7 +66,7 @@ void LastFilesSection::read(istream & is) && lastfiles.size() < num_lastfiles) lastfiles.push_back(file); else - LYXERR(Debug::INIT) << "LyX: Warning: Ignore last file: " << tmp << endl; + LYXERR(Debug::INIT, "LyX: Warning: Ignore last file: " << tmp); } while (is.good()); } @@ -97,7 +85,7 @@ void LastFilesSection::add(FileName const & file) LastFiles::iterator it = find(lastfiles.begin(), lastfiles.end(), file); if (it != lastfiles.end()) lastfiles.erase(it); - lastfiles.push_front(file); + lastfiles.insert(lastfiles.begin(), file); if (lastfiles.size() > num_lastfiles) lastfiles.pop_back(); } @@ -108,9 +96,8 @@ void LastFilesSection::setNumberOfLastFiles(unsigned int no) if (0 < no && no <= absolute_max_last_files) num_lastfiles = no; else { - LYXERR(Debug::INIT) << "LyX: session: too many last files\n" - << "\tdefault (=" << default_num_last_files - << ") used." << endl; + LYXERR(Debug::INIT, "LyX: session: too many last files\n" + << "\tdefault (=" << default_num_last_files << ") used."); num_lastfiles = default_num_last_files; } } @@ -124,14 +111,31 @@ void LastOpenedSection::read(istream & is) if (c == '[') break; getline(is, tmp); - if (tmp == "" || tmp[0] == '#' || tmp[0] == ' ' || !absolutePath(tmp)) + if (tmp.empty() || tmp[0] == '#' || tmp[0] == ' ') continue; - FileName const file(tmp); - if (file.exists() && !file.isDirectory()) - lastopened.push_back(file); - else - LYXERR(Debug::INIT) << "LyX: Warning: Ignore last opened file: " << tmp << endl; + try { + LastOpenedFile lof; + istringstream itmp(tmp); + itmp >> lof.active; + itmp.ignore(2); // ignore ", " + string fname; + getline(itmp, fname); + if (!FileName::isAbsolute(fname)) + continue; + + FileName const file(fname); + if (file.exists() && !file.isDirectory()) { + lof.file_name = file; + lastopened.push_back(lof); + } else { + LYXERR(Debug::INIT, + "LyX: Warning: Ignore last opened file: " << tmp); + } + } catch (...) { + LYXERR(Debug::INIT, + "LyX: Warning: unknown state of last opened file: " << tmp); + } } while (is.good()); } @@ -139,14 +143,25 @@ void LastOpenedSection::read(istream & is) void LastOpenedSection::write(ostream & os) const { os << '\n' << sec_lastopened << '\n'; - copy(lastopened.begin(), lastopened.end(), - ostream_iterator(os, "\n")); + for (size_t i = 0; i < lastopened.size(); ++i) + os << lastopened[i].active << ", " << lastopened[i].file_name << '\n'; } -void LastOpenedSection::add(FileName const & file) +void LastOpenedSection::add(FileName const & file, bool active) { - lastopened.push_back(file); + LastOpenedFile lof(file, active); + // check if file is already recorded (this can happen + // with multiple buffer views). We do only record each + // file once, since we cannot restore multiple views + // currently, we even crash in some cases (see #9483). + // FIXME: Add session support for multiple views of + // the same buffer (split-view etc.). + for (size_t i = 0; i < lastopened.size(); ++i) { + if (lastopened[i].file_name == file) + return; + } + lastopened.push_back(lof); } @@ -178,16 +193,16 @@ void LastFilePosSection::read(istream & is) itmp >> filepos.pos; itmp.ignore(2); // ignore ", " getline(itmp, fname); - if (!absolutePath(fname)) + if (!FileName::isAbsolute(fname)) continue; FileName const file(fname); if (file.exists() && !file.isDirectory() && lastfilepos.size() < num_lastfilepos) lastfilepos[file] = filepos; else - LYXERR(Debug::INIT) << "LyX: Warning: Ignore pos of last file: " << fname << endl; + LYXERR(Debug::INIT, "LyX: Warning: Ignore pos of last file: " << fname); } catch (...) { - LYXERR(Debug::INIT) << "LyX: Warning: unknown pos of last file: " << tmp << endl; + LYXERR(Debug::INIT, "LyX: Warning: unknown pos of last file: " << tmp); } } while (is.good()); } @@ -255,16 +270,16 @@ void BookmarksSection::read(istream & is) itmp >> pos; itmp.ignore(2); // ignore ", " getline(itmp, fname); - if (!absolutePath(fname)) + if (!FileName::isAbsolute(fname)) continue; FileName const file(fname); // only load valid bookmarks if (file.exists() && !file.isDirectory() && idx <= max_bookmarks) bookmarks[idx] = Bookmark(file, pit, pos, 0, 0); else - LYXERR(Debug::INIT) << "LyX: Warning: Ignore bookmark of file: " << fname << endl; + LYXERR(Debug::INIT, "LyX: Warning: Ignore bookmark of file: " << fname); } catch (...) { - LYXERR(Debug::INIT) << "LyX: Warning: unknown Bookmark info: " << tmp << endl; + LYXERR(Debug::INIT, "LyX: Warning: unknown Bookmark info: " << tmp); } } while (is.good()); } @@ -273,7 +288,7 @@ void BookmarksSection::read(istream & is) void BookmarksSection::write(ostream & os) const { os << '\n' << sec_bookmarks << '\n'; - for (size_t i = 1; i <= max_bookmarks; ++i) { + for (size_t i = 0; i <= max_bookmarks; ++i) { if (isValid(i)) os << i << ", " << bookmarks[i].bottom_pit << ", " @@ -299,103 +314,31 @@ bool BookmarksSection::isValid(unsigned int i) const } -BookmarksSection::Bookmark const & BookmarksSection::bookmark(unsigned int i) const +bool BookmarksSection::hasValid() const { - return bookmarks[i]; -} - - -void ToolbarSection::read(istream & is) -{ - string tmp; - do { - char c = is.peek(); - if (c == '[') - break; - getline(is, tmp); - if (tmp == "" || tmp[0] == '#' || tmp[0] == ' ') - continue; - - try { - // Read session info, saved as key/value pairs - // would better yell if pos returns npos - size_t pos = tmp.find_first_of(" = "); - // silently ignore lines without " = " - if (pos != string::npos) { - ToolbarItem item; - item.key = tmp.substr(0, pos); - int state; - int location; - istringstream value(tmp.substr(pos + 3)); - value >> state; - value >> location; - value >> item.info.posx; - value >> item.info.posy; - item.info.state = ToolbarInfo::State(state); - item.info.location = ToolbarInfo::Location(location); - toolbars.push_back(item); - } else - LYXERR(Debug::INIT) << "LyX: Warning: Ignore toolbar info: " << tmp << endl; - } catch (...) { - LYXERR(Debug::INIT) << "LyX: Warning: unknown Toolbar info: " << tmp << endl; - } - } while (is.good()); - // sort the toolbars by location, line and position - std::sort(toolbars.begin(), toolbars.end()); + for (size_t i = 1; i <= size(); ++i) { + if (isValid(i)) + return true; + } + return false; } -void ToolbarSection::write(ostream & os) const +BookmarksSection::Bookmark const & BookmarksSection::bookmark(unsigned int i) const { - os << '\n' << sec_toolbars << '\n'; - for (ToolbarList::const_iterator tb = toolbars.begin(); - tb != toolbars.end(); ++tb) { - os << tb->key << " = " - << static_cast(tb->info.state) << " " - << static_cast(tb->info.location) << " " - << tb->info.posx << " " - << tb->info.posy << '\n'; - } + return bookmarks[i]; } -ToolbarSection::ToolbarInfo & ToolbarSection::load(string const & name) +LastCommandsSection::LastCommandsSection(unsigned int num) : + default_num_last_commands(30), + absolute_max_last_commands(100) { - for (ToolbarList::iterator tb = toolbars.begin(); - tb != toolbars.end(); ++tb) - if (tb->key == name) - return tb->info; - - // add a new item - ToolbarItem item; - item.key = name; - toolbars.push_back(item); - return toolbars.back().info; + setNumberOfLastCommands(num); } - -bool operator<(ToolbarSection::ToolbarItem const & a, ToolbarSection::ToolbarItem const & b) -{ - ToolbarSection::ToolbarInfo lhs = a.info; - ToolbarSection::ToolbarInfo rhs = b.info; - // on if before off - if (lhs.state != rhs.state) - return static_cast(lhs.state) < static_cast(rhs.state); - // order of dock does not really matter - if (lhs.location != rhs.location) - return static_cast(lhs.location) < static_cast(rhs.location); - // if the same dock, the order depends on position - if (lhs.location == ToolbarSection::ToolbarInfo::TOP || - lhs.location == ToolbarSection::ToolbarInfo::BOTTOM) - return lhs.posy < rhs.posy || (lhs.posy == rhs.posy && lhs.posx < rhs.posx); - else if (lhs.location == ToolbarSection::ToolbarInfo::LEFT || - lhs.location == ToolbarSection::ToolbarInfo::RIGHT) - return lhs.posx < rhs.posx || (lhs.posx == rhs.posx && lhs.posy < rhs.posy); - return true; -} - - -void SessionInfoSection::read(istream & is) + +void LastCommandsSection::read(istream & is) { string tmp; do { @@ -406,58 +349,49 @@ void SessionInfoSection::read(istream & is) if (tmp == "" || tmp[0] == '#' || tmp[0] == ' ') continue; - try { - // Read session info, saved as key/value pairs - // would better yell if pos returns npos - string::size_type pos = tmp.find_first_of(" = "); - // silently ignore lines without " = " - if (pos != string::npos) { - string key = tmp.substr(0, pos); - string value = tmp.substr(pos + 3); - sessioninfo[key] = value; - } else - LYXERR(Debug::INIT) << "LyX: Warning: Ignore session info: " << tmp << endl; - } catch (...) { - LYXERR(Debug::INIT) << "LyX: Warning: unknown Session info: " << tmp << endl; - } + lastcommands.push_back(tmp); } while (is.good()); } -void SessionInfoSection::write(ostream & os) const +void LastCommandsSection::write(ostream & os) const +{ + os << '\n' << sec_lastcommands << '\n'; + copy(lastcommands.begin(), lastcommands.end(), + ostream_iterator(os, "\n")); +} + + +void LastCommandsSection::setNumberOfLastCommands(unsigned int no) { - os << '\n' << sec_session << '\n'; - for (MiscInfo::const_iterator val = sessioninfo.begin(); - val != sessioninfo.end(); ++val) { - os << val->first << " = " << val->second << '\n'; + if (0 < no && no <= absolute_max_last_commands) + num_lastcommands = no; + else { + LYXERR(Debug::INIT, "LyX: session: too many last commands\n" + << "\tdefault (=" << default_num_last_commands << ") used."); + num_lastcommands = default_num_last_commands; } } -void SessionInfoSection::save(string const & key, string const & value) +void LastCommandsSection::add(std::string const & string) { - sessioninfo[key] = value; + lastcommands.push_back(string); } -string const SessionInfoSection::load(string const & key, bool release) +void LastCommandsSection::clear() { - MiscInfo::const_iterator pos = sessioninfo.find(key); - string value; - if (pos != sessioninfo.end()) - value = pos->second; - if (release) - sessioninfo.erase(key); - return value; + lastcommands.clear(); } -Session::Session(unsigned int num) : - last_files(num) +Session::Session(unsigned int num_last_files, unsigned int num_last_commands) : + last_files(num_last_files), last_commands(num_last_commands) { // locate the session file // note that the session file name 'session' is hard-coded - session_file = FileName(addName(package().user_support().absFilename(), "session")); + session_file = FileName(addName(package().user_support().absFileName(), "session")); // readFile(); } @@ -484,12 +418,13 @@ void Session::readFile() lastFilePos().read(is); else if (tmp == sec_bookmarks) bookmarks().read(is); - else if (tmp == sec_toolbars) - toolbars().read(is); - else if (tmp == sec_session) - sessionInfo().read(is); + else if (tmp == sec_lastcommands) + lastCommands().read(is); + else if (tmp == sec_authfiles) + authFiles().read(is); + else - LYXERR(Debug::INIT) << "LyX: Warning: unknown Session section: " << tmp << endl; + LYXERR(Debug::INIT, "LyX: Warning: unknown Session section: " << tmp); } } @@ -504,12 +439,45 @@ void Session::writeFile() const lastFiles().write(os); lastOpened().write(os); lastFilePos().write(os); + lastCommands().write(os); bookmarks().write(os); - toolbars().write(os); - sessionInfo().write(os); + authFiles().write(os); } else - LYXERR(Debug::INIT) << "LyX: Warning: unable to save Session: " - << session_file << endl; + LYXERR(Debug::INIT, "LyX: Warning: unable to save Session: " + << session_file); } + +AuthFilesSection::AuthFilesSection() { } + + +void AuthFilesSection::read(istream & is) +{ + string tmp; + do { + char c = is.peek(); + if (c == '[') + break; + getline(is, tmp); + if (tmp.empty() || tmp[0] == '#' || tmp[0] == ' ' || !FileName::isAbsolute(tmp)) + continue; + + // read lastfiles + FileName const file(tmp); + if (file.exists() && !file.isDirectory()) + auth_files_.insert(tmp); + else + LYXERR(Debug::INIT, "LyX: Warning: Ignore auth file: " << tmp); + } while (is.good()); +} + + +void AuthFilesSection::write(ostream & os) const +{ + os << '\n' << sec_authfiles << '\n'; + copy(auth_files_.begin(), auth_files_.end(), + ostream_iterator(os, "\n")); +} + + }