On Mar 26, 2010, at 11:43 AM, Pieter de Bie wrote:
>> +
>> + if (n < 0) {
>> + if (errno == EINTR) {
>> + n = 1;
>> + continue;
>
> Why set n=1 here? Also, if you continue here..
>> + } else {
>
> you don't need else here.
The n=1 is for the while loop which tests (n>0). However that is no longer
needed, so I should get rid of it.
The else was left over from my previous patch which did not use continue and I
never changed it.
I'll clean up the code and resubmit the patch.
--Nathan
http://brotherbard.com/
To unsubscribe from this group, send email to gitx+unsubscribegooglegroups.com
or reply to this email with the words "REMOVE ME" as the subject.
