[PATCH] KMainWindow::applyMainWindowSettings & SaveMainWindowSettings

[Thomas Zander]

On Sat, Jun 08, 2002 at 12:10:32PM -0400, Nadeem Hasan wrote:
> On Saturday 08 June 2002 02:29 am, Thomas Zander wrote:
> <snip>
> > 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.
> 
> And why is that so important? What great performance or anything else does
> it buy you? You have given a completely bogus argument here.

hmm? I'm not attacking you, and you seem to go into defend...
Anyway, you snipped away your statement mine above replied to: (I added indents
for convience)

> > > 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.
So my answer:
> > 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.

Just tries to tell you your concerns are not something that is a problem
right now.

I'm afraid that if you want to change the behavior of something _you_ are
going to have to give a reason why to change it. Your questions toward me
don't seem to go into that direction..


> <snip>
> 
> > > 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?
> 
> So you mean you don't have a problem with the existing docs?
> 

Did I say that? I wonder where I said that..  I'm just asking you to write 
some docs for an API you seem to have been looking into.

I'm just asking, if you don't want to do that, then  someone else should. 
The problem with that is it will most probably be forgotten. 

So "pretty please" write some docs :)

Thanx!

-- 
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/bb08e867/attachment.sig>