Hi all,

Well, first of all. I want to thank you all your pacience answering my questions. But now there is a piece of code that I cannot understand, or that I misunderstand. The code is kopete 0.12.2 (trying to search the problem with the picture that doesn't load if the contact is not using Live Messenger). The file is kopetecontactlistview.cpp and the class is ArrangeByGroupsViewStrategy or ArrangeByPresenceViewStrategy.

I cannot understand why do you search 'for people in groups' instead of finding 'groups in people'. For example in updateMetaContactInner from ArrangeByPresenceViewStrategy class does 2 "for" loops (not nested), for searching anybody into the connected or disconnected groups. This is crazy or at least I cannot understand it.

Why don't add a new attribute to the metacontact that saves in which group/s is the metacontact ?

Think about it... what is better ? A user in group or in not a lot of groups, or a lot of users in not a lot of groups.

If we had 100 users added, and 5 groups. You are doing lots of loops, and they could be lots less !!

Thank you,
Rafael Fernández López.
_______________________________________________
kopete-devel mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/kopete-devel

Reply via email to