Patch to make KNotify look in apps' own sound dirs

Neil Stevens neil at qualityassistant.com
Fri Oct 11 02:02:55 BST 2002


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

On Thursday October 10, 2002 04:24, Carsten Pfeiffer wrote:
> On Thursday 10 October 2002 10:54, Neil Stevens wrote:
>
> Hiya,
>
> > Right now apps have to install their sounds into $(kde_sounddir) for
> > knotify to find their sounds.  This patch lets sounds in
> > $(kde_datadir)/<appname>/sounds be found, too.
>
> shouldn't it look in the appsdir first and then in the global dir?

Yes.  Good catch.

> And
> isn't findResource() a bit more efficient (if you only want the first
> file anyway).

You're right.

> And finally, can you add some bits of documentation about this to
> KNotifyClient?

Yes.

Attached are the KNotify and the KNotifyClient parts of the second try.

- -- 
Neil Stevens - neil at qualityassistant.com
"The nearest I can make it out, 'Love your Enemies' means, 'Hate your
Friends'." - Benjamin Franklin
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.6 (GNU/Linux)
Comment: For info see http://www.gnupg.org

iD8DBQE9piM/f7mnligQOmERAkq5AJ9spIBvQ9tZeEjaspOcdQiJlCCLlgCfYlAa
k0vLKcFPRfWmYryssirIZx8=
=1VyZ
-----END PGP SIGNATURE-----
-------------- next part --------------
A non-text attachment was scrubbed...
Name: knotifyclient
Type: text/x-diff
Size: 866 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20021010/98fb2be4/attachment.diff>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: knotify
Type: text/x-diff
Size: 1840 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20021010/98fb2be4/attachment-0001.diff>


More information about the kde-core-devel mailing list