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