[PATCH] Accelerator saving

Waldo Bastian bastian at kde.org
Thu Apr 1 13:44:20 BST 2004


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

On Tue March 30 2004 23:21, Rob Kaper wrote:
> The saving and re-loading of accelerators breaks if the app's ui.rc file
> has a comment as a sibling of the ActionProperties element before that
> element. The attached patch by Neil should fix that, though we didn't get a
> chance to test, since we just use packaged kdelibs instead of building our
> own these days.
>
> Can someone test in HEAD and commit and possibly backport?

Slightly modified, tested & confirmed with kreversi and commited to both HEAD 
and KDE_3_2_BRANCH.

This line was funny:
	// KXMLGUIFactory::removeDOMComments( it ); <-- What was this for? --ellis

Nasty bug, thanks for the patch.

Cheers,
Waldo
- -- 
>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>
^ bastian at kde.org | Is your software SUSE LINUX READY? | bastian at suse.com
^<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.2 (GNU/Linux)

iD8DBQFAbA6kN4pvrENfboIRAmEnAJ9bvY5HYeNAxAH/5RQfkxjJ4/t82gCgnRuN
QAtvlsv+fVW+8xsdaCxnWIg=
=rvdG
-----END PGP SIGNATURE-----




More information about the kde-core-devel mailing list