FUJITA Tomonori wrote:
> On Fri, 18 Jul 2008 18:37:24 +0200
> Albert Pauw <[EMAIL PROTECTED]> wrote:
>
>
>> Just a small patch.
>>
>> The wrong return value was checked in the if statement.
>>
>> Albert
>>
>> --------
>>
>> --- bs_ssc.c 2008-07-17 19:19:10.000000000 +0200
>> +++ bs_ssc.c.new 2008-07-18 18:34:15.000000000 +0200
>> @@ -65,7 +65,7 @@
>> case REZERO_UNIT:
>> rew = lseek(fd, 0, SEEK_SET);
>> curr_pos = lseek(fd, 0, SEEK_CUR);
>> - if (ret)
>> + if (rew)
>> set_medium_error(&result, &key, &asc);
>> eprintf("Rewind Successful, File Pointer at %" PRIu64",%m\n",
>> curr_pos);
>>
>
> I think that the fix is right but what's for the second lseek? Can we
> remove it?
>
>
I believe the second seek is put in to be used only by the eprintf
function, just to check the
current position and show that the seek was succesfull.
Since it is still work in progress and anything near finished I would
leave it in for the moment.
Albert
_______________________________________________
Stgt-devel mailing list
[email protected]
https://lists.berlios.de/mailman/listinfo/stgt-devel
