comparison libpurple/connection.c @ 20119:ec3471ab96d6

Check in purple_connection_error_reason that purple_connection_reason_is_fatal == wants_to_die, and log a debug warning if not.

20118:c26b7aff4e96

purple_connection_error_reason (PurpleConnection *gc,
                                PurpleDisconnectReason reason,
                                const char *description)
{
	PurpleConnectionUiOps *ops;

	g_return_if_fail(gc   != NULL);

	if (description == NULL) {
		purple_debug_error("connection", "purple_connection_error_reason: check `description != NULL' failed\n");
		description = _("Unknown error");
	}

	g_assert (reason < PURPLE_NUM_REASONS);

	/* If we've already got one error, we don't need any more */
	if (gc->disconnect_timeout)
		return;

20119:ec3471ab96d6

purple_connection_error_reason (PurpleConnection *gc,
                                PurpleDisconnectReason reason,
                                const char *description)
{
	PurpleConnectionUiOps *ops;
	gboolean fatal;

	g_return_if_fail(gc   != NULL);

	if (description == NULL) {
		purple_debug_error("connection", "purple_connection_error_reason: check `description != NULL' failed\n");
		description = _("Unknown error");
	}

	g_assert (reason < PURPLE_NUM_REASONS);

	/* This should probably be removed at some point */
	fatal = purple_connection_reason_is_fatal (reason);
	if (fatal != gc->wants_to_die)
		purple_debug_warning ("connection",
			"reason %u is %sfatal but wants_to_die is %u",
			reason, (fatal ? "" : "not "), gc->wants_to_die);

	/* If we've already got one error, we don't need any more */
	if (gc->disconnect_timeout)
		return;