[Konsole-devel] Review Request: Get rid of the _defaultProfile member of MainWindow
Jekyll Wu
adaptee at gmail.com
Sun Jan 29 13:00:09 UTC 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103819/
-----------------------------------------------------------
Review request for Konsole.
Description
-------
_defaultProfile was previously used to record the profile to be used for the "New Tab" and the "New Window" action. In other words, it is the actual default profile in practice.
Now that "New Tab" and "New Window" actions always use the default profile, there is not much need to keep that member anymore. It is actually a little confusing to have both the concept of applicatioin-wide default profile and the concept of window-wide default profile.
Diffs
-----
src/Application.h 7415fd5
src/Application.cpp 646cd4b
src/MainWindow.h 2fb1803
src/MainWindow.cpp 7908394
Diff: http://git.reviewboard.kde.org/r/103819/diff/diff
Testing
-------
I checked the "-e", "--profile", "-p", and "--tabs-from-file" options. No regression observed so far.
Thanks,
Jekyll Wu
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20120129/1463f884/attachment.html>
More information about the konsole-devel
mailing list