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
