On 04/05/2009 23:29, Vincent van Ravesteijn wrote:
Abdelrazak Younes schreef:
On 04/05/2009 19:49, Vincent van Ravesteijn wrote:
Hi all,
This patch adjusts the session file to contain which file was last
opened. It'll look like this:
[last opened files]
0, C:/Users/Vincent/Documents/newfile36.lyx
1, C:/Users/Vincent/Documents/newfile34.lyx
0, C:/Users/Vincent/Documents/newfile33.lyx
0, C:/Users/Vincent/Documents/newfile30.lyx
Ok ?
not for me, sorry. I would like to kill this session class...
Abdel.
meaning ...
... that you are going to kill the session class, also in branch,
My plan is to kill it in trunk only. Your patch is fine for branch.
within a certain timeframe (1.6.3 hint hint), or
... you don't want to fix the bug, or
That too yes :-)
... you don't like the way it is fixed (and you know an alternative) ?
For trunk? It's not that I don't like the logic, it's just the
implementation that I don't like. Session handling is a GUI thing, it
has nothing to do in src/
So, if I had the time (this is already a far fetched idea at this
point), I would transfer the remaining bit of the Session class to the
relevant classes using QSettings. GuiView for example should take care
of the last _viewed_ files (not necessarily all last _opened_ files)
onto this window. WorkArea would take care of last cursor position, etc.
Abdel.
