pidgin: 74295d33: Wow, this isn't used. That will save us...
markdoliner at pidgin.im
markdoliner at pidgin.im
Thu May 29 13:45:43 EDT 2008
-----------------------------------------------------------------
Revision: 74295d3392f7f386b2178a9c8b571226b6a25c60
Ancestor: 4604a1b4f5a0af87939613abf115bb25b4a7c97f
Author: markdoliner at pidgin.im
Date: 2008-05-29T17:41:08
Branch: im.pidgin.pidgin
URL: http://d.pidgin.im/viewmtn/revision/info/74295d3392f7f386b2178a9c8b571226b6a25c60
Modified files:
libpurple/status.c
ChangeLog:
Wow, this isn't used. That will save us either 4 bytes or 8 bytes for
each PurpleStatus, depending on whether you're on a 32 bit or 64 bit
OS. We create a full set of PurpleStatuses (between 2 and maybe 10,
depending on the protocol) for each PurplePresence, and each buddy in
your buddylist has a PurplePresence.
Assuming you on a 64 bit machine, you have 200 buddies, and you're
using a protocol with 8 PurpleStatusTypes, this saves you 12KB of
memory. Eh, I guess that's not so bad.
Our status system is pretty memory inefficient right now... we tend
to duplicate a lot of stuff from the PurpleStatusTypes unneccessarily.
I think we can probably get a way with NOT duplicating it and having
PurplePresence and PurpleStatus look through to the PurpleStatusTypes
as appropriate. But it'll break our API a little, so we can't do it
until 3.0.0.
-------------- next part --------------
============================================================
--- libpurple/status.c 25a57059f1557a35cfcafb10e16edc7006e85b91
+++ libpurple/status.c 2ee76141aed0a328a1438ed67a818bda7a350ffb
@@ -107,8 +107,6 @@ struct _PurpleStatus
PurpleStatusType *type;
PurplePresence *presence;
- const char *title;
-
gboolean active;
/*
More information about the Commits
mailing list