Video effects

Matthias Kretz kretz at kde.org
Fri Jul 13 13:46:11 CEST 2007


On Friday 13 July 2007, Thierry Bastian wrote:
> The problem is still the same: we can't easily have a common base of
> effects. So adding a GaussianBlurEffect class would not make sense out of
> the Quicktime world.
>
> Thinking loud: Maybe a factory could do that job?

That's why I was rather conservative with the specialized effect classes I 
defined. I've only added a class if I had a valid use case for an application 
that needs the class.

How would a factory help? All it would change API wise is from:
GaussianBlurEffect *e = new GaussianBlurEffect(parent);
if (e->isValid()) {
    ...
}

to:
GaussianBlurEffect *e = new Factory::create(GaussianBlurEffect, parent);
if (e) {
    ...
}

or:
Effect *e = new Factory::create(GaussianBlurEffect, parent);
if (e) {
    ...
}

-- 
________________________________________________________
Matthias Kretz (Germany)                            <><
http://Vir.homelinux.org/
MatthiasKretz at gmx.net, kretz at kde.org,
Matthias.Kretz at urz.uni-heidelberg.de
-------------- 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/phonon-backends/attachments/20070713/0a69b4b7/attachment.pgp 


More information about the Phonon-backends mailing list