# HG changeset patch # User Mark Doliner # Date 1215478701 0 # Node ID 6dc33eb841aa33761dffeb4ce8a0f65471f6a5a0 # Parent 1fcdc7d11f38ad9641ef09967136286508b6d19b Remove a failsafe check that I don't believe is needed. I'm under the impression that sometimes the last modified timestamp will (at least somewhat) legitimately be set to 0 in cases where we really have received our roster. And this function doesn't seem to get called when we haven't received our roster. So, uh, this should fix the aim buddylist for a very small fraction of users who probably gave up on using Pidgin when it didn't work for them the first time they tried. diff -r 1fcdc7d11f38 -r 6dc33eb841aa libpurple/protocols/oscar/oscar.c --- a/libpurple/protocols/oscar/oscar.c Mon Jul 07 01:03:44 2008 +0000 +++ b/libpurple/protocols/oscar/oscar.c Tue Jul 08 00:58:21 2008 +0000 @@ -4928,11 +4928,6 @@ purple_debug_info("oscar", "ssi: syncing local list and server list\n"); - if ((timestamp == 0) || (numitems == 0)) { - purple_debug_info("oscar", "Got AIM SSI with a 0 timestamp or 0 numitems--not syncing. This probably means your buddy list is empty.\n"); - return 1; - } - /* Clean the buddy list */ aim_ssi_cleanlist(od);