[Kde-games-devel] KSirk Jabber code

[Kleag]

Albert,

Yes, I talked to the kopete Jabber plugin developer (I put Olivier in copy). 
If I remember well, he said that the iris lib in kopete was an old version and 
that if I could use a newer version, it would be better (am I right, Olivier 
?). Then I adapted the latest iris lib and I planned to do the job to adapt 
existing kopete code to be able to use it too. But to try my ideas quickly, I 
started by copying the relevant part of the kopete Jabber plugin. In fact, I'm 
not using in kopete a lot of this code features.

Also, I feeled that it would be not very efficient to ask the adition of 
something that should go into kdelibs where the intersection was in fact 
limited and also because it was dependent on an external, iris, even if iris 
is directly integrated in the kopete plugin.

Finaly, I planned to port the kopete Jabber plugin to use the new iris 
version, and I actualy started the job, but the summer was finished and I 
could not find the time to do it cleanly in time...

So, what do you think I should do right now ? Knowing that the Jabber 
integration in KsirK is still experimental and will need some work to make it 
completely usable (a warning inform the user about that) du to speed reasons 
at game start. 

Regards,

Gaël
Le Wednesday 10 December 2008 23:25:00 Albert Astals Cid, vous avez écrit :
> It's a copy of kdenetwork/kopete/protocols/jabber
>
> That's bad, if you have a look almost all files already have minor/major
> changes.
>
> Gaël have you tried talking to the kopete guys to factorize the code into a
> library you both can use? How much code of all this are you really using?
>
> Albert
>
> _______________________________________________
> kde-games-devel mailing list
> kde-games-devel at kde.org
> https://mail.kde.org/mailman/listinfo/kde-games-devel

-- 
KsirK - a world domination strategy game 
http://techbase.kde.org/Projects/Games/Tactic_and_Strategy/KsirK

KGraphViewer - a GraphViz dot graphs viewer
http://extragear.kde.org/apps/kgraphviewer