On Thu, 2004-10-21 at 21:02 -0400, Colin Walters wrote: > + if (!auth_finished) > + if (!do_reading (transport)) > + { I think braces around the if() body would make this clearer. So, I like this patch more than the other one. If you agree, go ahead and commit, and thanks for the fix. Havoc