BIC Problem with KPlayObjectFactory resolved

[Neil Stevens]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Thursday August 29, 2002 12:56, Matthias Welwarsky wrote:
> On Wednesday 28 August 2002 19:07, Neil Stevens wrote:
> > Aha!  So can you call the new one KStreamingPlayObjectFactory?  That
> > way people trying to figure out this stuff by reading the docs will
> > know which is which.
>
> Hm, we will have to stick with this name, and in fact the creator can
> create both, streaming and nonstreaming objects.

Please make sure to add documentation to both classes then, both the 
Factory and the Creator, so that the already existing ignorance in KDE 
multimedia programming isn't compounded.

> However, Stefan Westerfeld had an alternative Idea: KPlayObjectFactory
> could internally use a KPlayObjectCreator, and have an additional
> createPlayObject-method like this one:
>
> bool KPlayObjectFactory::createPlayObject(const KURL& url, bool
> createBUS, QObject* receiver, const char* slot);
>
> It would be called like this:
>
> factory.createPlayObject(url, createBUS, receiverObject,
> SLOT(receiverSlot(KPlayObject*)));
>
> So, we wouldn't need to change the inheritance of KPlayObjectFactory,
> because we'd just connect PlayObjectCreators signal to the slot
> delivered by the caller...
>
> However, we'd need an additional pointer to the Creator as a member
> variable, and I don't see how I can do this binary compatible, as I must
> not change the size of the factory objects...

Unfortunate that it was forgotten to add a d pointer to the 
KPlayObjectFactory.  Make sure to add one to the Creator. :-)

Can the Creator be made a "singleton," storing its own pointer?  That 
should allow the Factory to use a Creator without problems.

- -- 
Neil Stevens - neil at qualityassistant.com
"I always cheer up immensely if an attack is particularly wounding
because I think, well, if they attack one personally, it means they
have not a single political argument left." - Margaret Thatcher
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.6 (GNU/Linux)
Comment: For info see http://www.gnupg.org

iD8DBQE9bn7uf7mnligQOmERAl+/AJ40Z3G8ekMXdTIqurhhh0I6GAlWzACgoV1y
lPhIB5J7KAh2cNE+MF10Bt8=
=UjDn
-----END PGP SIGNATURE-----