about a patch
richardmg at trolltech.com
Tue Nov 25 15:25:07 CET 2008
>> Then you can:
> Yes, which should be treated as the application doing something
> wrong (not
> fatal, though), whether we want to handle it the same as
> I'm not sure about yet.
Q_ASSERT(source.type() == Empty);
mediaObject->setCurrentSource(source); // Remove currently loaded media.
...would make sense to me (then there would be not need for the static
function, alltough I don't think it so bad).
> There's no MediaObject::setCurrentSource(MediaSource::Type) function
> and no
> MediaSource(MediaSource::Type) ctor atm.
>> I still don't see the need for
> We need to either add
> static MediaSource MediaSource::empty()
> I'd rather add the latter than the former because e.g.
> would then be possible but doesn't make any sense. I do agree that
> is more intuitive than
> So it's a trade-off between making the API foolproof and making it
> Matthias Kretz (Germany) <><
> Phonon-backends mailing list
> Phonon-backends at kde.org
More information about the Phonon-backends