Tue, 22 Jun 2010 22:44:28 +0000
Destroy Parts when the SlpMessage is destroyed.
| 8810 | 1 | /** |
| 2 | * @file transaction.c MSN transaction functions | |
| 3 | * | |
| 15884 | 4 | * purple |
| 8810 | 5 | * |
| 15884 | 6 | * Purple is the legal property of its developers, whose names are too numerous |
|
9198
e8eb6d5eb9eb
[gaim-migrate @ 9993]
Christian Hammond <chipx86@chipx86.com>
parents:
9193
diff
changeset
|
7 | * to list here. Please refer to the COPYRIGHT file distributed with this |
|
e8eb6d5eb9eb
[gaim-migrate @ 9993]
Christian Hammond <chipx86@chipx86.com>
parents:
9193
diff
changeset
|
8 | * source distribution. |
| 8810 | 9 | * |
| 10 | * This program is free software; you can redistribute it and/or modify | |
| 11 | * it under the terms of the GNU General Public License as published by | |
| 12 | * the Free Software Foundation; either version 2 of the License, or | |
| 13 | * (at your option) any later version. | |
| 14 | * | |
| 15 | * This program is distributed in the hope that it will be useful, | |
| 16 | * but WITHOUT ANY WARRANTY; without even the implied warranty of | |
| 17 | * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | |
| 18 | * GNU General Public License for more details. | |
| 19 | * | |
| 20 | * You should have received a copy of the GNU General Public License | |
| 21 | * along with this program; if not, write to the Free Software | |
|
19859
71d37b57eff2
The FSF changed its address a while ago; our files were out of date.
John Bailey <rekkanoryo@rekkanoryo.org>
parents:
15884
diff
changeset
|
22 | * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02111-1301 USA |
| 8810 | 23 | */ |
|
30945
b875cf477e19
Remove unnecesary includes from msn.h.
Jorge Villaseñor <masca@cpw.pidgin.im>
parents:
30915
diff
changeset
|
24 | |
|
30961
885064b16c54
Include internal.h on each c file to avoid windows breakage as recommended by Daniel.
Jorge Villaseñor <masca@cpw.pidgin.im>
parents:
30945
diff
changeset
|
25 | #include "internal.h" |
|
30945
b875cf477e19
Remove unnecesary includes from msn.h.
Jorge Villaseñor <masca@cpw.pidgin.im>
parents:
30915
diff
changeset
|
26 | #include "debug.h" |
|
b875cf477e19
Remove unnecesary includes from msn.h.
Jorge Villaseñor <masca@cpw.pidgin.im>
parents:
30915
diff
changeset
|
27 | |
| 8810 | 28 | #include "msn.h" |
| 29 | #include "transaction.h" | |
| 30 | ||
| 31 | MsnTransaction * | |
|
10225
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
32 | msn_transaction_new(MsnCmdProc *cmdproc, const char *command, |
|
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
33 | const char *format, ...) |
| 8810 | 34 | { |
| 35 | MsnTransaction *trans; | |
| 36 | va_list arg; | |
| 37 | ||
| 38 | g_return_val_if_fail(command != NULL, NULL); | |
| 39 | ||
| 40 | trans = g_new0(MsnTransaction, 1); | |
| 41 | ||
|
10225
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
42 | trans->cmdproc = cmdproc; |
| 8810 | 43 | trans->command = g_strdup(command); |
|
30915
56295f0a8ade
I don't really like this name.
Elliott Sales de Andrade <qulogic@pidgin.im>
parents:
30910
diff
changeset
|
44 | trans->saveable = TRUE; |
| 8810 | 45 | |
|
9158
f8dab42adeaf
[gaim-migrate @ 9942]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
8810
diff
changeset
|
46 | if (format != NULL) |
|
f8dab42adeaf
[gaim-migrate @ 9942]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
8810
diff
changeset
|
47 | { |
|
f8dab42adeaf
[gaim-migrate @ 9942]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
8810
diff
changeset
|
48 | va_start(arg, format); |
|
f8dab42adeaf
[gaim-migrate @ 9942]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
8810
diff
changeset
|
49 | trans->params = g_strdup_vprintf(format, arg); |
|
f8dab42adeaf
[gaim-migrate @ 9942]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
8810
diff
changeset
|
50 | va_end(arg); |
|
f8dab42adeaf
[gaim-migrate @ 9942]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
8810
diff
changeset
|
51 | } |
| 8810 | 52 | |
| 53 | /* trans->queue = g_queue_new(); */ | |
|
9198
e8eb6d5eb9eb
[gaim-migrate @ 9993]
Christian Hammond <chipx86@chipx86.com>
parents:
9193
diff
changeset
|
54 | |
| 8810 | 55 | return trans; |
| 56 | } | |
| 57 | ||
| 58 | void | |
| 59 | msn_transaction_destroy(MsnTransaction *trans) | |
| 60 | { | |
| 61 | g_return_if_fail(trans != NULL); | |
| 62 | ||
| 63 | g_free(trans->command); | |
| 64 | g_free(trans->params); | |
| 65 | g_free(trans->payload); | |
| 66 | ||
|
28649
95d25c1e195d
Allow setting a destroy-function for the data. Fixes a leak.
Sadrul Habib Chowdhury <sadrul@pidgin.im>
parents:
25583
diff
changeset
|
67 | if (trans->data_free) |
|
95d25c1e195d
Allow setting a destroy-function for the data. Fixes a leak.
Sadrul Habib Chowdhury <sadrul@pidgin.im>
parents:
25583
diff
changeset
|
68 | trans->data_free(trans->data); |
|
95d25c1e195d
Allow setting a destroy-function for the data. Fixes a leak.
Sadrul Habib Chowdhury <sadrul@pidgin.im>
parents:
25583
diff
changeset
|
69 | |
| 8810 | 70 | #if 0 |
| 71 | if (trans->pendent_cmd != NULL) | |
| 72 | msn_message_unref(trans->pendent_msg); | |
| 73 | #endif | |
| 74 | ||
| 75 | #if 0 | |
| 76 | MsnTransaction *elem; | |
| 77 | if (trans->queue != NULL) | |
|
9198
e8eb6d5eb9eb
[gaim-migrate @ 9993]
Christian Hammond <chipx86@chipx86.com>
parents:
9193
diff
changeset
|
78 | { |
| 8810 | 79 | while ((elem = g_queue_pop_head(trans->queue)) != NULL) |
| 80 | msn_transaction_destroy(elem); | |
|
9198
e8eb6d5eb9eb
[gaim-migrate @ 9993]
Christian Hammond <chipx86@chipx86.com>
parents:
9193
diff
changeset
|
81 | |
| 8810 | 82 | g_queue_free(trans->queue); |
| 83 | } | |
| 84 | #endif | |
| 85 | ||
|
10284
257978b5df5b
[gaim-migrate @ 11454]
Stu Tomlinson <nosnilmot@pidgin.im>
parents:
10225
diff
changeset
|
86 | if (trans->callbacks != NULL && trans->has_custom_callbacks) |
|
257978b5df5b
[gaim-migrate @ 11454]
Stu Tomlinson <nosnilmot@pidgin.im>
parents:
10225
diff
changeset
|
87 | g_hash_table_destroy(trans->callbacks); |
|
257978b5df5b
[gaim-migrate @ 11454]
Stu Tomlinson <nosnilmot@pidgin.im>
parents:
10225
diff
changeset
|
88 | |
|
10225
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
89 | if (trans->timer) |
| 15884 | 90 | purple_timeout_remove(trans->timer); |
|
10225
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
91 | |
| 8810 | 92 | g_free(trans); |
| 93 | } | |
| 94 | ||
| 95 | char * | |
| 96 | msn_transaction_to_string(MsnTransaction *trans) | |
| 97 | { | |
| 98 | char *str; | |
| 99 | ||
| 100 | g_return_val_if_fail(trans != NULL, FALSE); | |
| 101 | ||
| 102 | if (trans->params != NULL) | |
| 103 | str = g_strdup_printf("%s %u %s\r\n", trans->command, trans->trId, trans->params); | |
|
30915
56295f0a8ade
I don't really like this name.
Elliott Sales de Andrade <qulogic@pidgin.im>
parents:
30910
diff
changeset
|
104 | else if (trans->saveable) |
|
30910
374c457d65b5
We need a way to send commands without transaction ID
Jorge Villaseñor <masca@cpw.pidgin.im>
parents:
28649
diff
changeset
|
105 | str = g_strdup_printf("%s %u\r\n", trans->command, trans->trId); |
| 8810 | 106 | else |
|
30910
374c457d65b5
We need a way to send commands without transaction ID
Jorge Villaseñor <masca@cpw.pidgin.im>
parents:
28649
diff
changeset
|
107 | str = g_strdup_printf("%s\r\n", trans->command); |
| 8810 | 108 | |
| 109 | return str; | |
| 110 | } | |
| 111 | ||
| 112 | void | |
|
9193
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
113 | msn_transaction_queue_cmd(MsnTransaction *trans, MsnCommand *cmd) |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
114 | { |
| 15884 | 115 | purple_debug_info("msn", "queueing command.\n"); |
|
9193
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
116 | trans->pendent_cmd = cmd; |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
117 | msn_command_ref(cmd); |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
118 | } |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
119 | |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
120 | void |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
121 | msn_transaction_unqueue_cmd(MsnTransaction *trans, MsnCmdProc *cmdproc) |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
122 | { |
|
9218
6ebf0fd0df6b
[gaim-migrate @ 10014]
Stu Tomlinson <nosnilmot@pidgin.im>
parents:
9198
diff
changeset
|
123 | MsnCommand *cmd; |
|
6ebf0fd0df6b
[gaim-migrate @ 10014]
Stu Tomlinson <nosnilmot@pidgin.im>
parents:
9198
diff
changeset
|
124 | |
|
10481
a5d6b8e1717d
[gaim-migrate @ 11769]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
10403
diff
changeset
|
125 | if (!cmdproc->servconn->connected) |
|
a5d6b8e1717d
[gaim-migrate @ 11769]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
10403
diff
changeset
|
126 | return; |
|
a5d6b8e1717d
[gaim-migrate @ 11769]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
10403
diff
changeset
|
127 | |
| 15884 | 128 | purple_debug_info("msn", "unqueueing command.\n"); |
|
9218
6ebf0fd0df6b
[gaim-migrate @ 10014]
Stu Tomlinson <nosnilmot@pidgin.im>
parents:
9198
diff
changeset
|
129 | cmd = trans->pendent_cmd; |
|
6ebf0fd0df6b
[gaim-migrate @ 10014]
Stu Tomlinson <nosnilmot@pidgin.im>
parents:
9198
diff
changeset
|
130 | |
|
9193
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
131 | g_return_if_fail(cmd != NULL); |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
132 | |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
133 | msn_cmdproc_process_cmd(cmdproc, cmd); |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
134 | msn_command_unref(cmd); |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
135 | |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
136 | trans->pendent_cmd = NULL; |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
137 | } |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
138 | |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
139 | #if 0 |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
140 | void |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
141 | msn_transaction_queue(MsnTransaction *trans, MsnTransaction *elem) |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
142 | { |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
143 | if (trans->queue == NULL) |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
144 | trans->queue = g_queue_new(); |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
145 | |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
146 | g_queue_push_tail(trans->queue, elem); |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
147 | } |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
148 | |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
149 | void |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
150 | msn_transaction_unqueue(MsnTransaction *trans, MsnCmdProc *cmdproc) |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
151 | { |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
152 | MsnTransaction *elem; |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
153 | |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
154 | while ((elem = g_queue_pop_head(trans->queue)) != NULL) |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
155 | msn_cmdproc_send_trans(cmdproc, elem); |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
156 | } |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
157 | #endif |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
158 | |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
159 | void |
| 8810 | 160 | msn_transaction_set_payload(MsnTransaction *trans, |
| 161 | const char *payload, int payload_len) | |
| 162 | { | |
| 163 | g_return_if_fail(trans != NULL); | |
| 164 | g_return_if_fail(payload != NULL); | |
| 165 | ||
| 166 | trans->payload = g_strdup(payload); | |
| 167 | trans->payload_len = payload_len ? payload_len : strlen(trans->payload); | |
| 168 | } | |
|
9193
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
169 | |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
170 | void |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
171 | msn_transaction_set_data(MsnTransaction *trans, void *data) |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
172 | { |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
173 | g_return_if_fail(trans != NULL); |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
174 | |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
175 | trans->data = data; |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
176 | } |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
177 | |
|
28649
95d25c1e195d
Allow setting a destroy-function for the data. Fixes a leak.
Sadrul Habib Chowdhury <sadrul@pidgin.im>
parents:
25583
diff
changeset
|
178 | void msn_transaction_set_data_free(MsnTransaction *trans, GDestroyNotify fn) |
|
95d25c1e195d
Allow setting a destroy-function for the data. Fixes a leak.
Sadrul Habib Chowdhury <sadrul@pidgin.im>
parents:
25583
diff
changeset
|
179 | { |
|
95d25c1e195d
Allow setting a destroy-function for the data. Fixes a leak.
Sadrul Habib Chowdhury <sadrul@pidgin.im>
parents:
25583
diff
changeset
|
180 | g_return_if_fail(trans != NULL); |
|
95d25c1e195d
Allow setting a destroy-function for the data. Fixes a leak.
Sadrul Habib Chowdhury <sadrul@pidgin.im>
parents:
25583
diff
changeset
|
181 | trans->data_free = fn; |
|
95d25c1e195d
Allow setting a destroy-function for the data. Fixes a leak.
Sadrul Habib Chowdhury <sadrul@pidgin.im>
parents:
25583
diff
changeset
|
182 | } |
|
95d25c1e195d
Allow setting a destroy-function for the data. Fixes a leak.
Sadrul Habib Chowdhury <sadrul@pidgin.im>
parents:
25583
diff
changeset
|
183 | |
|
9193
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
184 | void |
|
30915
56295f0a8ade
I don't really like this name.
Elliott Sales de Andrade <qulogic@pidgin.im>
parents:
30910
diff
changeset
|
185 | msn_transaction_set_saveable(MsnTransaction *trans, gboolean saveable) |
|
30910
374c457d65b5
We need a way to send commands without transaction ID
Jorge Villaseñor <masca@cpw.pidgin.im>
parents:
28649
diff
changeset
|
186 | { |
|
374c457d65b5
We need a way to send commands without transaction ID
Jorge Villaseñor <masca@cpw.pidgin.im>
parents:
28649
diff
changeset
|
187 | g_return_if_fail(trans != NULL); |
|
374c457d65b5
We need a way to send commands without transaction ID
Jorge Villaseñor <masca@cpw.pidgin.im>
parents:
28649
diff
changeset
|
188 | |
|
30915
56295f0a8ade
I don't really like this name.
Elliott Sales de Andrade <qulogic@pidgin.im>
parents:
30910
diff
changeset
|
189 | trans->saveable = saveable; |
|
30910
374c457d65b5
We need a way to send commands without transaction ID
Jorge Villaseñor <masca@cpw.pidgin.im>
parents:
28649
diff
changeset
|
190 | } |
|
374c457d65b5
We need a way to send commands without transaction ID
Jorge Villaseñor <masca@cpw.pidgin.im>
parents:
28649
diff
changeset
|
191 | |
|
374c457d65b5
We need a way to send commands without transaction ID
Jorge Villaseñor <masca@cpw.pidgin.im>
parents:
28649
diff
changeset
|
192 | void |
|
9193
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
193 | msn_transaction_add_cb(MsnTransaction *trans, char *answer, |
|
10225
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
194 | MsnTransCb cb) |
|
9193
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
195 | { |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
196 | g_return_if_fail(trans != NULL); |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
197 | g_return_if_fail(answer != NULL); |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
198 | |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
199 | if (trans->callbacks == NULL) |
|
10284
257978b5df5b
[gaim-migrate @ 11454]
Stu Tomlinson <nosnilmot@pidgin.im>
parents:
10225
diff
changeset
|
200 | { |
|
257978b5df5b
[gaim-migrate @ 11454]
Stu Tomlinson <nosnilmot@pidgin.im>
parents:
10225
diff
changeset
|
201 | trans->has_custom_callbacks = TRUE; |
|
257978b5df5b
[gaim-migrate @ 11454]
Stu Tomlinson <nosnilmot@pidgin.im>
parents:
10225
diff
changeset
|
202 | trans->callbacks = g_hash_table_new_full(g_str_hash, g_str_equal, NULL, |
|
257978b5df5b
[gaim-migrate @ 11454]
Stu Tomlinson <nosnilmot@pidgin.im>
parents:
10225
diff
changeset
|
203 | NULL); |
|
257978b5df5b
[gaim-migrate @ 11454]
Stu Tomlinson <nosnilmot@pidgin.im>
parents:
10225
diff
changeset
|
204 | } |
|
257978b5df5b
[gaim-migrate @ 11454]
Stu Tomlinson <nosnilmot@pidgin.im>
parents:
10225
diff
changeset
|
205 | else if (trans->has_custom_callbacks != TRUE) |
|
257978b5df5b
[gaim-migrate @ 11454]
Stu Tomlinson <nosnilmot@pidgin.im>
parents:
10225
diff
changeset
|
206 | g_return_if_reached (); |
|
9193
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
207 | |
|
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
208 | g_hash_table_insert(trans->callbacks, answer, cb); |
|
10225
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
209 | } |
|
9193
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
210 | |
|
10225
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
211 | static gboolean |
|
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
212 | transaction_timeout(gpointer data) |
|
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
213 | { |
|
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
214 | MsnTransaction *trans; |
|
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
215 | |
|
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
216 | trans = data; |
|
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
217 | g_return_val_if_fail(trans != NULL, FALSE); |
|
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
218 | |
|
10403
4647a87ef73b
[gaim-migrate @ 11648]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
10284
diff
changeset
|
219 | #if 0 |
| 15884 | 220 | purple_debug_info("msn", "timed out: %s %d %s\n", trans->command, trans->trId, trans->params); |
|
10403
4647a87ef73b
[gaim-migrate @ 11648]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
10284
diff
changeset
|
221 | #endif |
|
10225
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
222 | |
|
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
223 | if (trans->timeout_cb != NULL) |
|
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
224 | trans->timeout_cb(trans->cmdproc, trans); |
|
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
225 | |
|
25583
4f63a60ea58b
Set this timer to 0 when we return FALSE
Mark Doliner <markdoliner@pidgin.im>
parents:
24619
diff
changeset
|
226 | trans->timer = 0; |
|
10225
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
227 | return FALSE; |
|
9193
6e76f1367896
[gaim-migrate @ 9988]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9158
diff
changeset
|
228 | } |
|
10225
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
229 | |
|
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
230 | void |
|
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
231 | msn_transaction_set_timeout_cb(MsnTransaction *trans, MsnTimeoutCb cb) |
|
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
232 | { |
|
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
233 | if (trans->timer) |
|
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
234 | { |
| 15884 | 235 | purple_debug_error("msn", "This shouldn't be happening\n"); |
| 236 | purple_timeout_remove(trans->timer); | |
|
10225
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
237 | } |
|
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
238 | trans->timeout_cb = cb; |
|
24619
5ece4c87ea96
Use purple_timeout_add_seconds() instead of purple_timeout_add() in a
Mark Doliner <markdoliner@pidgin.im>
parents:
19859
diff
changeset
|
239 | trans->timer = purple_timeout_add_seconds(60, transaction_timeout, trans); |
|
10225
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
240 | } |
|
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
241 | |
|
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
242 | void |
|
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
243 | msn_transaction_set_error_cb(MsnTransaction *trans, MsnErrorCb cb) |
|
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
244 | { |
|
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
245 | trans->error_cb = cb; |
|
0dfea1bc8695
[gaim-migrate @ 11357]
Felipe Contreras <felipe.contreras@gmail.com>
parents:
9218
diff
changeset
|
246 | } |