[kdebase trunk] medianotifier: replace Q3CString with QByteArray

[Kevin Ottens]

Le Mercredi 26 Octobre 2005 05:41, Dave Rowe a écrit :
> Attached is patch replacing Q3CString with QByteArray for the following
> files:
>
> kioslave/media/medianotifier/medianotifier.cpp
> kioslave/media/medianotifier/medianotifier.h
>
> Please review and reply to me with any comments or suggestions!

Looks sane to me.

You can remove the following comment though:
+// Replacing Q3CString with QByteArray as noted
+// in the Trolltech documentation

It doesn't really makes sense to keep such a comment in the current context 
IMHO.

Oh, and please be careful you screwed up my name in the header. ;-)
Seems to be an encoding issue.

Regards.
-- 
Kévin 'ervin' Ottens, http://ervin.ipsquad.net
"Ni le maître sans disciple, Ni le disciple sans maître,
Ne font reculer l'ignorance."
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20051026/c891a678/attachment.sig>