PATCH: Fix for BR#48679: Proxy options lost after switching proxy use off and on

Dawit A. adawit at kde.org
Fri Nov 8 04:44:10 GMT 2002


On Thursday 07 November 2002 05:03, David Faure wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> kenvvarproxydlg.diff :
> - - setMinimumSize looks wrong and/or unnecessary

I fixed it another way.  Perhaps this is more acceptable for now without 
redoing the layout at this point ?  Note the problem was that toggling the 
button caused the layout to expand and contract.  In hind sight I could have 
perhaps held it off until post 3.1...

> - - an i18n string is changed! Don't apply that!

Hmm... I thought this was okay if the same string already exists in the same 
module/app, no ?  That string is repeated in ::showValue (....).  Removing 
the unnecessary spaces from "Hide &Value" would indeed break i18n.

I found another problem however.  A shortcut conflict in KEnvVarProxyDlg.  I 
fixed it in the patch attached below, but I am sure that breaks i18n for the 
modified string as well.

> - - textChanged() copies the text into the lineedits below. This is in fact
> another way to 'lose' information (if you type 3 different proxies, then
> click the "same for all" checkbox by accident, and uncheck it -> you lose
> the settings in the two lineedits).
>
> kmanualproxy.diff:
> - - same objection, it seems the patch is mostly about copying values in
> the "same for all" case....

Fixed.  Please try the attached patch.

> ksaveioconfig.diff

> - - one minor thing: setObject(0) is unnecessary, KStaticDeleter resets the
> static pointer already.

OK.

> - - Of course we could use KStaticDeleter<KConfig> directly, instead of the
> intermediary class KSaveIOConfigPrivate, but indeed the approach used by
> the patch is open for more extensions in the future.
:)

> - - I see a good memleak fix in updateRunningIOSlaves.
Complaint or complement ?  I presume the former.  Could not tell from the tone 
:)

> - - Why does updateConfiguration delete "d" ? That sounds dangerous, given
> that it's the object in the kstaticdeleter! I would delete (and set to 0)  
d->config instead. I'll make that change and commit that one.

Hmm... I fail to see why since we set it to 0 in case of double deletion.   
Howerver, I am not sure how KStaticDeleter works and consider your approach 
to be better than mine ; so I have no objections.  

On the other hand KProtocolManager does the same thing except that the object 
maintained by the static deleter has a couple of more pointers.  Should that 
be modified as well ?

Regards,
Dawit A.





More information about the kfm-devel mailing list