Mercurial > pidgin.yaz
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 |
rev | line source |
---|---|
12838 | 1 /** @page gtklog-signals GtkLog Signals |
2 | |
3 @signals | |
4 @signal log-displaying | |
5 @endsignals | |
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 | 9 <hr> |
10 | |
11 @signaldef log-displaying | |
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 | 14 @endsignalproto |
15 @signaldesc | |
16 Emitted when a log is being displayed. | |
17 @param viewer The log viewer | |
18 @param log The log being displayed | |
19 @endsignaldef | |
20 | |
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 |