pidgin: decc53d7: Remove PurpleConnectionUiOps.report_disc...

markdoliner at pidgin.im markdoliner at pidgin.im
Sun Aug 21 04:08:48 EDT 2011


----------------------------------------------------------------------
Revision: decc53d7ccf47667a39c4af96677e68b2b6c1a08
Parent:   ce6903fbc56ae18f649789c1f774cc214e71169d
Author:   markdoliner at pidgin.im
Date:     08/21/11 03:34:02
Branch:   im.pidgin.pidgin
URL: http://d.pidgin.im/viewmtn/revision/info/decc53d7ccf47667a39c4af96677e68b2b6c1a08

Changelog: 

Remove PurpleConnectionUiOps.report_disconnect

Changes against parent ce6903fbc56ae18f649789c1f774cc214e71169d

  patched  ChangeLog.API
  patched  finch/gntconn.c
  patched  libpurple/connection.c
  patched  libpurple/connection.h
  patched  pidgin/gtkconn.c

-------------- next part --------------
============================================================
--- libpurple/connection.c	e26d259b085aa54418b2ddd5f4e01b1ed087c441
+++ libpurple/connection.c	fe71528d0f9455fbfa498cac43d04397b6c6f753
@@ -578,13 +578,8 @@ purple_connection_error_reason (PurpleCo
 
 	ops = purple_connections_get_ui_ops();
 
-	if (ops != NULL)
-	{
-		if (ops->report_disconnect_reason != NULL)
-			ops->report_disconnect_reason (gc, reason, description);
-		if (ops->report_disconnect != NULL)
-			ops->report_disconnect (gc, description);
-	}
+	if (ops && ops->report_disconnect_reason)
+		ops->report_disconnect_reason(gc, reason, description);
 
 	purple_signal_emit(purple_connections_get_handle(), "connection-error",
 		gc, reason, description);
============================================================
--- libpurple/connection.h	951499e579935f8e24497d558ca22b912eb46d5e
+++ libpurple/connection.h	64edb9cbc7a4a25883ea3551147476d805d7bcad
@@ -194,16 +194,6 @@ typedef struct
 	void (*notice)(PurpleConnection *gc, const char *text);
 
 	/**
-	 * Called when an error causes a connection to be disconnected.
-	 * Called before #disconnected.
-	 * @param text  a localized error message.
-	 * @see #purple_connection_error
-	 * @deprecated in favour of
-	 *             #PurpleConnectionUiOps.report_disconnect_reason.
-	 */
-	void (*report_disconnect)(PurpleConnection *gc, const char *text);
-
-	/**
 	 * Called when libpurple discovers that the computer's network
 	 * connection is active.  On Linux, this uses Network Manager if
 	 * available; on Windows, it uses Win32's network change notification
@@ -219,17 +209,15 @@ typedef struct
 
 	/**
 	 * Called when an error causes a connection to be disconnected.
-	 *  Called before #disconnected.  This op is intended to replace
-	 *  #report_disconnect.  If both are implemented, this will be called
-	 *  first; however, there's no real reason to implement both.
+	 * Called before #disconnected.
 	 *
-	 *  @param reason  why the connection ended, if known, or
-	 *                 #PURPLE_CONNECTION_ERROR_OTHER_ERROR, if not.
-	 *  @param text  a localized message describing the disconnection
-	 *               in more detail to the user.
-	 *  @see #purple_connection_error_reason
+	 * @param reason  why the connection ended, if known, or
+	 *                #PURPLE_CONNECTION_ERROR_OTHER_ERROR, if not.
+	 * @param text  a localized message describing the disconnection
+	 *              in more detail to the user.
+	 * @see #purple_connection_error_reason
 	 *
-	 *  @since 2.3.0
+	 * @since 2.3.0
 	 */
 	void (*report_disconnect_reason)(PurpleConnection *gc,
 	                                 PurpleConnectionError reason,
============================================================
--- pidgin/gtkconn.c	03e3f77c4205f96e2156f994175896d60374bee1
+++ pidgin/gtkconn.c	e8fcd836f7029a5f3e975dd4f26e3a8c9da469dd
@@ -215,7 +215,6 @@ static PurpleConnectionUiOps conn_ui_ops
 	pidgin_connection_connected,
 	pidgin_connection_disconnected,
 	pidgin_connection_notice,
-	NULL, /* report_disconnect */
 	pidgin_connection_network_connected,
 	pidgin_connection_network_disconnected,
 	pidgin_connection_report_disconnect_reason,
============================================================
--- ChangeLog.API	878a4ef74d00a5cbc708d22a7ed3f3703b4b2d4f
+++ ChangeLog.API	fc99fe2417f6b99a2c54d0dcf1c2584bd4b2cf27
@@ -13,12 +13,15 @@ version 3.0.0 (??/??/????):
 		  the first parameter
 
 		Removed:
+		* GtkIMHtml.clipboard_html_string
+		* GtkIMHtml.clipboard_text_string
 		* pidgin_blist_update_account_error_state
 		* pidgin_check_if_dir
 		* PIDGIN_DIALOG
 		* pidgin_dialogs_alias_contact
 		* pidgin_set_custom_buddy_icon
 		* pidgin_setup_screenname_autocomplete
+		* PidginConversation.sg
 		* purple_core_migrate
 		* purple_notify_searchresults_column_get_title
 		* purple_notify_searchresults_get_columns_count
@@ -30,10 +33,8 @@ version 3.0.0 (??/??/????):
 		* purple_strlcpy
 		* purple_util_fetch_url_request_len_with_account.  Use
 		  purple_util_fetch_url_request_len, insetad.
+		* PurpleConnectionUiOps.report_disconnect
 		* struct _GtkIMHtmlFontDetail
-		* The 'clipboard_text_string' and 'clipboard_html_string' members of
-		  GtkIMHtml
-		* The 'sg' member of PidginConversation
 
 version 2.10.0:
 	libpurple:
============================================================
--- finch/gntconn.c	da06c5fb762d7cf3fb8d9a8d932259f2a43127c3
+++ finch/gntconn.c	1d35589732b6df3da1490beacf7606e2e313005b
@@ -165,7 +165,6 @@ static PurpleConnectionUiOps ops =
 	NULL, /* connected */
 	NULL, /* disconnected */
 	NULL, /* notice */
-	NULL,
 	NULL, /* network_connected */
 	NULL, /* network_disconnected */
 	finch_connection_report_disconnect,


More information about the Commits mailing list