[PATCH] kopenwith.cpp bugfix and enhancement

[David Faure]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Monday 10 February 2003 19:48, Willy De la Court wrote:
> 
> ChangeLog
> ------------------------------
> added new checkbox "Do not &close when command exits" to the "Open With..." 
> dialog this checkbox is only visible when using konsole as set in the 
> Component chooser.
> the checkbox is disabled until the "Run in Terminal" is activated
> 
> added the --noclose to the TerminalOptions var but only if the 
> TerminalApplication is konsole

Hmm, I'm not overly happy with one more checkbox in that dialog, for the
sake of its usability, but it does sound like a very useful feature indeed.

> fixed a bug so that Terminal is set to true when "Run in Terminal" is checked

? Can you explain that one? You seem to have moved the code, but
it does the same as before about this, doesn't it?

> instead of adding the --noclose to the TerminalOptions I could add a new 
> option in the .desktop file but since there seems to be a block on changes 
> for .desktop files (standard) this is the best option
You can add new X-KDE-Foo fields in .desktop files, that's no problem. 
But I prefer it to be in terminaloptions, since that's what it really is....

> also this would mean changes in the krun.cpp but for the moment i want to stay 
> away from changes in something so critical
:)

> the patch is fully functional and tested
> can someone with write access to the cvs  commit this patch please
> or if you don't agree with the patch please let me know
The patch isn't binary compatible, since you added a member variable to KOpenWithDlg.
You'll need to define the Private class used by the d pointer and move it there....
Hmm, a better idea: remove the QPushButton* ok; member variable instead,
it's not used.
WOW!
I removed it, and it shows that the while (!ok) on line 556 was using it instead
of the bool. ARGL. This might explain a few bugs :}
Ok I'll fix that and commit your patch as well.
Thanks.

- -- 
David Faure -- faure at kde.org, dfaure at klaralvdalens-datakonsult.se
Klarälvdalens Datakonsult AB, Platform-independent software solutions
Contributing to: http://www.konqueror.org/, http://www.koffice.org/
KOffice-1.2.1 is available - http://download.kde.org/stable/koffice-1.2.1/
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.7 (GNU/Linux)

iD8DBQE+SPDB72KcVAmwbhARAq8yAJ90JdyUHHxnc2/EcLxDPcIVXeI+pwCcCa/G
Z9WV+MLy2TtN8pi8SUi0FgM=
=Uu9p
-----END PGP SIGNATURE-----