[PATCH] KMainWindow::applyMainWindowSettings & SaveMainWindowSettings

Thomas Zander 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
are near.
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
Type: application/pgp-signature
Size: 232 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20020608/65e9135e/attachment.sig>

More information about the kde-core-devel mailing list