about a patch
Matthias Kretz
kretz at kde.org
Thu Nov 27 15:35:30 CET 2008
On Wednesday 26 November 2008 09:04:13 Richard wrote:
> Just one minor thing, I would perhaps rephrase:
> + * Creates a MediaSource that empties the current source.
> + * It can be used to reset the MediaObject into the state it
> has before setting the first source.
>
> into something more like this:
>
> + * Creates an empty MediaSource. An empty MediaSource is
> considered valid, and can be set on a
> + * MediaObject to unload its current media.
>
> since the constructor does not actually empty or modify 'the current'
> source anywhere. But it can be used to empty/unload the current media
> from a particular MediaObject.
>
> (I would also suggest something similar for: + * This MediaSource
> empties the current source.)
done, thanks.
--
________________________________________________________
Matthias Kretz (Germany) <><
http://Vir.homelinux.org/
More information about the Phonon-backends
mailing list