pidgin/pidgin

Stop automatically adding/remove conversations from the conversation manager

7 weeks ago, Gary Kramlich
8019945e1c21
Stop automatically adding/remove conversations from the conversation manager

This was done to mimic the old conversations API and shouldn't be needed going
forward. However, since we can't create conversations right now, we're probably
missing some edge cases here.

Also make sure to keep a reference around on the conversation while removing to avoid use after frees.

Testing Done:
Trained with the turtles.

Reviewed at https://reviews.imfreedom.org/r/3049/
/*
* Pidgin - Internet Messenger
* Copyright (C) Pidgin Developers <devel@pidgin.im>
*
* Pidgin is the legal property of its developers, whose names are too numerous
* to list here. Please refer to the COPYRIGHT file distributed with this
* source distribution.
*
* This program is free software; you can redistribute it and/or modify
* it under the terms of the GNU General Public License as published by
* the Free Software Foundation; either version 2 of the License, or
* (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*
* You should have received a copy of the GNU General Public License
* along with this program; if not, see <https://www.gnu.org/licenses/>.
*/
#include "pidginiconname.h"
/******************************************************************************
* Public API
*****************************************************************************/
const gchar *
pidgin_icon_name_from_status_primitive(PurpleStatusPrimitive primitive,
const gchar *fallback)
{
switch(primitive) {
case PURPLE_STATUS_OFFLINE:
return "pidgin-user-offline";
break;
case PURPLE_STATUS_AVAILABLE:
return "pidgin-user-available";
break;
case PURPLE_STATUS_AWAY:
return "pidgin-user-away";
break;
case PURPLE_STATUS_INVISIBLE:
return "pidgin-user-invisible";
break;
case PURPLE_STATUS_UNAVAILABLE:
case PURPLE_STATUS_EXTENDED_AWAY:
return "pidgin-user-do-not-disturb";
break;
case PURPLE_STATUS_MOBILE:
case PURPLE_STATUS_UNSET:
default:
break;
}
return fallback;
}
const gchar *
pidgin_icon_name_from_status_type(PurpleStatusType *type, const gchar *fallback)
{
PurpleStatusPrimitive primitive;
if(type == NULL) {
return fallback;
}
primitive = purple_status_type_get_primitive(type);
return pidgin_icon_name_from_status_primitive(primitive, fallback);
}
const gchar *
pidgin_icon_name_from_status(PurpleStatus *status, const gchar *fallback) {
PurpleStatusType *type = NULL;
if(!PURPLE_IS_STATUS(status)) {
return fallback;
}
type = purple_status_get_status_type(status);
if(type == NULL) {
return fallback;
}
return pidgin_icon_name_from_status_type(type, fallback);
}
const char *
pidgin_icon_name_from_presence_primitive(PurplePresencePrimitive primitive,
const char *fallback)
{
switch(primitive) {
case PURPLE_PRESENCE_PRIMITIVE_OFFLINE:
return "pidgin-user-offline";
break;
case PURPLE_PRESENCE_PRIMITIVE_AVAILABLE:
return "pidgin-user-available";
break;
case PURPLE_PRESENCE_PRIMITIVE_IDLE:
return "pidgin-user-unavailable";
break;
case PURPLE_PRESENCE_PRIMITIVE_INVISIBLE:
return "pidgin-user-invisible";
break;
case PURPLE_PRESENCE_PRIMITIVE_AWAY:
return "pidgin-user-away";
break;
case PURPLE_PRESENCE_PRIMITIVE_DO_NOT_DISTURB:
return "pidgin-user-do-not-disturb";
break;
case PURPLE_PRESENCE_PRIMITIVE_STREAMING:
/* TODO: get an icon for streaming. */
default:
break;
}
return fallback;
}
const char *
pidgin_icon_name_from_presence(PurplePresence *presence, const char *fallback)
{
PurplePresencePrimitive primitive;
primitive = purple_presence_get_primitive(presence);
return pidgin_icon_name_from_presence_primitive(primitive, fallback);
}