libpurple/protocols/jabber/parser.h@acdb4dbab299 (annotated)
libpurple/protocols/jabber/parser.h
Wed, 21 Jan 2009 00:19:33 +0000
- author
- Paul Aurich <darkrain42@pidgin.im>
- date
- Wed, 21 Jan 2009 00:19:33 +0000
- branch
- cpw.darkrain42.xmpp.bosh
- changeset 25488
- acdb4dbab299
- parent 19897
- 8b5abbdebc87
- child 25726
- 19e0c9302a43
- child 26703
- 17f9a4bef2a3
- permissions
- -rw-r--r--
Don't track a cb-per-POST and remove the PurpleHTTPResponse structure.
The only callback ever used is http_received_cb and the ordering of
responses from the server is not guaranteed to match the order of our requests,
so the metaphor of matching them doesn't make sense. Instead of that, just
track the number of requests (to ensure there is always a request outstanding).
Additionally, pass the data const-ified instead of copying it. It's just
fed to an XML parser anyway.
UPDATE:
Tobias pointed out my explanation was flat-out wrong. The server's replies *do*
correspond to a specific request (HTTP pipelining). I believe what I meant to
say is that the XML in the replies isn't guaranteed to correspond to the
request's payload.
In any event, the critical point is that the callback is always the same, so
there's no need to store it per-request.
| 7014 | 1 | /** |
| 2 | * @file parser.h XML parser functions | |
| 3 | * | |
| 15884 | 4 | * purple |
| 7014 | 5 | * |
| 6 | * Copyright (C) 2003 Nathan Walp <faceprint@faceprint.com> | |
| 7 | * | |
| 8 | * This program is free software; you can redistribute it and/or modify | |
| 9 | * it under the terms of the GNU General Public License as published by | |
| 10 | * the Free Software Foundation; either version 2 of the License, or | |
| 11 | * (at your option) any later version. | |
| 12 | * | |
| 13 | * This program is distributed in the hope that it will be useful, | |
| 14 | * but WITHOUT ANY WARRANTY; without even the implied warranty of | |
| 15 | * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | |
| 16 | * GNU General Public License for more details. | |
| 17 | * | |
| 18 | * You should have received a copy of the GNU General Public License | |
| 19 | * 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
|
20 | * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02111-1301 USA |
| 7014 | 21 | */ |
| 15884 | 22 | #ifndef _PURPLE_JABBER_PARSER_H_ |
| 23 | #define _PURPLE_JABBER_PARSER_H_ | |
| 7014 | 24 | |
| 25 | #include "jabber.h" | |
| 26 | ||
| 27 | void jabber_parser_setup(JabberStream *js); | |
|
17810
c8d4297080cb
Fixed a leak: The XML parser was never cleaned up on disconnect.
Andreas Monitzer <am@adiumx.com>
parents:
15884
diff
changeset
|
28 | void jabber_parser_free(JabberStream *js); |
| 7014 | 29 | void jabber_parser_process(JabberStream *js, const char *buf, int len); |
| 30 | ||
| 15884 | 31 | #endif /* _PURPLE_JABBER_PARSER_H_ */ |
