From: SourceForge.net <no...@so...> - 2005-06-01 01:50:39
|
Patches item #1197934, was opened at 2005-05-08 20:38 Message generated for change (Settings changed) made by lschiere You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1197934&group_id=235 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 5 Submitted By: Richard Laager (rlaager) >Assigned to: Mark Doliner (thekingant) Summary: IRC File Descriptor Check Initial Comment: Gaim always prints IRC QUIT messages to the console. This patch corrects that. This patch will cause problems if an IRC socket ever uses file descriptor zero. On Linux, I don't think that's possible. I'm not sure on other operating systems. ---------------------------------------------------------------------- Comment By: Richard Laager (rlaager) Date: 2005-05-08 22:45 Message: Logged In: YES user_id=156487 (21:22:48) Caleb Gilmour (hyphenated): rlaager: wouldn't it be nicer if you changed sslconn.c so that it does a gsc->fd = -1; in there after the g_new0 ? (it'd require more changes, but 0 isn't really appropriate for describing an undefined fd) (21:34:04) rlaager: hyphenated: I don't think that's required for the SSL stuff, but you have a good point. I'm redoing the patch to use that approach for IRC. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1197934&group_id=235 |