annotate doc/gtklog-signals.dox @ 23995:85bed17fe5c1

The variable we use to keep track of the watcher of the ssl connection should be unsigned. This isn't really a problem in Pidgin, where we use glib's mainloop and GIOChannels because glib starts assigning the handle IDs sequentially starting from 1. But if an eventloop implementation ever returns a handle ID greater than the largest possible signed integer (2,147,483,647) then we won't be able to remove the watcher because purple_ssl_close() in sslconn.c only removes it if inpa > 0, and since it interprets inpa as a signed value then handles over 2,147,483,647 appear as negative numbers. I stumbled upon this when playing around with libevent, which can use epoll. My implementation generated a random handle ID which was sometimes greater than 2,147,483,647. I don't believe this breaks binary compatibility. And I don't think it breaks source compatibility, but I guess it might depend on what compiler you're using.
author Mark Doliner <mark@kingant.net>
date Thu, 04 Sep 2008 18:04:29 +0000
parents e0613cf8c493
children
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
12838
cb36bc2db7c5 [gaim-migrate @ 15186]
Richard Laager <rlaager@wiktel.com>
parents:
diff changeset
1 /** @page gtklog-signals GtkLog Signals
cb36bc2db7c5 [gaim-migrate @ 15186]
Richard Laager <rlaager@wiktel.com>
parents:
diff changeset
2
cb36bc2db7c5 [gaim-migrate @ 15186]
Richard Laager <rlaager@wiktel.com>
parents:
diff changeset
3 @signals
cb36bc2db7c5 [gaim-migrate @ 15186]
Richard Laager <rlaager@wiktel.com>
parents:
diff changeset
4 @signal log-displaying
cb36bc2db7c5 [gaim-migrate @ 15186]
Richard Laager <rlaager@wiktel.com>
parents:
diff changeset
5 @endsignals
cb36bc2db7c5 [gaim-migrate @ 15186]
Richard Laager <rlaager@wiktel.com>
parents:
diff changeset
6
20807
e0613cf8c493 Add some links from signal documentation back to the documentation for the
Will Thompson <will.thompson@collabora.co.uk>
parents: 16183
diff changeset
7 @see gtklog.h
e0613cf8c493 Add some links from signal documentation back to the documentation for the
Will Thompson <will.thompson@collabora.co.uk>
parents: 16183
diff changeset
8
12838
cb36bc2db7c5 [gaim-migrate @ 15186]
Richard Laager <rlaager@wiktel.com>
parents:
diff changeset
9 <hr>
cb36bc2db7c5 [gaim-migrate @ 15186]
Richard Laager <rlaager@wiktel.com>
parents:
diff changeset
10
cb36bc2db7c5 [gaim-migrate @ 15186]
Richard Laager <rlaager@wiktel.com>
parents:
diff changeset
11 @signaldef log-displaying
cb36bc2db7c5 [gaim-migrate @ 15186]
Richard Laager <rlaager@wiktel.com>
parents:
diff changeset
12 @signalproto
16183
8cf53d7a0887 Update the Doxygen signals documentation to match the new struct names.
Richard Laager <rlaager@wiktel.com>
parents: 12838
diff changeset
13 void (*log_displaying)(PidginLogViewer *viewer, PurpleLog *log);
12838
cb36bc2db7c5 [gaim-migrate @ 15186]
Richard Laager <rlaager@wiktel.com>
parents:
diff changeset
14 @endsignalproto
cb36bc2db7c5 [gaim-migrate @ 15186]
Richard Laager <rlaager@wiktel.com>
parents:
diff changeset
15 @signaldesc
cb36bc2db7c5 [gaim-migrate @ 15186]
Richard Laager <rlaager@wiktel.com>
parents:
diff changeset
16 Emitted when a log is being displayed.
cb36bc2db7c5 [gaim-migrate @ 15186]
Richard Laager <rlaager@wiktel.com>
parents:
diff changeset
17 @param viewer The log viewer
cb36bc2db7c5 [gaim-migrate @ 15186]
Richard Laager <rlaager@wiktel.com>
parents:
diff changeset
18 @param log The log being displayed
cb36bc2db7c5 [gaim-migrate @ 15186]
Richard Laager <rlaager@wiktel.com>
parents:
diff changeset
19 @endsignaldef
cb36bc2db7c5 [gaim-migrate @ 15186]
Richard Laager <rlaager@wiktel.com>
parents:
diff changeset
20
cb36bc2db7c5 [gaim-migrate @ 15186]
Richard Laager <rlaager@wiktel.com>
parents:
diff changeset
21 */
20807
e0613cf8c493 Add some links from signal documentation back to the documentation for the
Will Thompson <will.thompson@collabora.co.uk>
parents: 16183
diff changeset
22 // vim: syntax=c.doxygen tw=75 et