[Konsole-devel] Review Request: Adds support for SHELL_SESSION_ID

[argonel at gmail.com]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/3045/
-----------------------------------------------------------

(Updated 2010-03-05 06:25:39.574585)


Review request for Konsole.


Changes
-------

Changed the "Uuid" session file key to "SessionGuid" to make it a little more clear as to what the value is for. It is not called "SHELL_SESSION_ID" as it is not the exact same representation of the QUuid as you would find in the shell.


Summary
-------

Adds support for SHELL_SESSION_ID, re-purposing the apparently unused Session::_uniqueIdentifier as a QUuid.

Session data appears to be stored via KMainWindow::saveGlobalProperties instead of saveProperties, and I think this causes the Uuid field to be copied between like-numbered sessions when one instance has a Uuid and one does not.

Thoughts?


This addresses bug 156563.
    https://bugs.kde.org/show_bug.cgi?id=156563


Diffs (updated)
-----

  /trunk/KDE/kdebase/apps/konsole/src/Session.h 1079359 
  /trunk/KDE/kdebase/apps/konsole/src/Session.cpp 1079359 

Diff: http://reviewboard.kde.org/r/3045/diff


Testing
-------


Thanks,

argonel