Wow. I need more coffee, I think. I got one of the two, at least. 50%
isn't bad?
> Matt Lilley wrote:
>>> Btw, it doesn't make sense to have while(1) immediately preceded by a
>>> conditional break IMO.
>> Agreed, changed.
> ..
>> +++ b/src/session.h
>> @@ -53,15 +53,16 @@
> ..
>> - if(rc) \
>> - break; \
>> - } while(1)
> ..
>> + } while(rc); \
> I think you inverted the logic. Add the ! I wrote. :)
>
>
> //Peter
> _______________________________________________
> libssh2-devel http://cool.haxx.se/cgi-bin/mailman/listinfo/libssh2-devel
>
-- _____________________________________________ Matt Lilley Software Engineer SecuritEase Tel: +64 4 912-2100 Fax: +64 4 912-2101 E-mail: matt.lilley_at_securitease.com Web: http://www.securitease.com _____________________________________________ This e-mail has passed our content security scan. It is covered by the confidentiality clauses at http://www.securitease.com/content_and_confidentialityReceived on 2011-05-03