LRN wrote:

>> Maybe you mean some other commit? For example,
>> <http://git.xiph.org/?p=flac.git;a=commitdiff;h=d66f6754bf94bc8ba23d3579d0b5650cd380c9f0>
>>  ?
> Yes, that is exactly what i meant.
>
>>
>> The attached patch *should* resolve the issue. libFLAC will call 
>> setvbuf(file, ...)
>> only if GetFileType(...file...) == FILE_TYPE_DISK.
>
> That patch looks correct. That said, i'm not sure why there's setvbuf() in
> the first place. I mean, the code to de-fragment the output file already
> exists (the aforementioned d66f675), so why buffering? Was it proven
> empirically that buffering is required? Or am i looking at this backwards
> and that code was added later than setvbuf()?

SetFilePointerEx() for _de_coding (e.g. when flac writes .wav file),
setvbuf() for _en_coding (when libFLAC writes .flac file).
_______________________________________________
flac-dev mailing list
flac-dev@xiph.org
http://lists.xiph.org/mailman/listinfo/flac-dev

Reply via email to