Mercurial > pidgin
changeset 12277:739e3aca067a
[gaim-migrate @ 14581]
Readding the error checking that I took out in the last commit. I thought about this more today and just confirmed.... It ensures correct behavior if the UI doesn't implement this part of the notify API (and it's not required to). Bartosz Oler agreed that I was right when presented with this case. I'm sorry for the indecision on this.
committer: Tailor Script <tailor@pidgin.im>
| author | Richard Laager <rlaager@wiktel.com> |
|---|---|
| date | Thu, 01 Dec 2005 00:36:19 +0000 |
| parents | b3652193d359 |
| children | 25d7ef365dbf |
| files | src/protocols/gg/gg.c |
| diffstat | 1 files changed, 10 insertions(+), 0 deletions(-) [+] |
line wrap: on
line diff
--- a/src/protocols/gg/gg.c Wed Nov 30 22:52:44 2005 +0000 +++ b/src/protocols/gg/gg.c Thu Dec 01 00:36:19 2005 +0000 @@ -1038,6 +1038,16 @@ (GaimNotifyCloseCallback)ggp_sr_close_cb, gaim_connection_get_account(gc)); + if (h == NULL) { + gaim_debug_error("gg", "ggp_pubdir_reply_handler: " + "Unable to display the search results.\n"); + gaim_notify_error(gc, NULL, + _("Unable to display the search results."), + NULL); + ggp_sr_close_cb(gaim_connection_get_account(gc)); + return; + } + info->searchresults_window = h; } else { gaim_notify_searchresults_new_rows(gc, results,