[PATCH] KMainWindow::applyMainWindowSettings & SaveMainWindowSettings
zander at planescape.com
Sat Jun 8 07:29:11 BST 2002
On Fri, Jun 07, 2002 at 06:31:17PM -0400, Nadeem Hasan wrote:
> On Friday 07 June 2002 06:43 pm, Waldo Bastian wrote:
> > On Friday 07 June 2002 02:42 pm, Nadeem Hasan wrote:
> > > Hi,
> > >
> > > If the group name is not passed to the above two methods, the group that
> > > used to read/write settings is completely random and results in the
> > > settings not getting restored properly. The attached patch defaults the
> > > group to "MainWindowSettings" to make the behaviour predictable.
> > The behaviour is already predictable. If no group is specified it uses the
> > current group. Changing that behaviour would break backwards compatibility.
> That would mean the app developer has to make sure that the same group is set
> before calling these methods. Not very nice IMO, as it may break the app very
> easily once the code starts to get shuffled around.
My experience was not like that; in the contrary. This behavior makes you
save all stuff in one group in one go. That means the setGroup kind of calls
I don't see this as a problem.
> > > BTW, why is the "group" parameter documented as "For internal use only"?
> > I have no idea. I suggest to update the documentation to include that the
> > settings are saved to "group" and that if "group" is empty the current
> > group is used.
> Yes, I think this *should* be done to make it clear.
Could you write something so it is clear to you? And provide a patch?
Thomas Zander zander at planescape.com
We are what we pretend to be
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 232 bytes
Desc: not available
More information about the kde-core-devel