[FFmpeg-devel] [PATCH] make libavcodec use bytestream functions

Ramiro Ribeiro Polla ramiro
Thu May 10 20:48:42 CEST 2007


Hi,

M?ns Rullg?rd wrote:
> Ramiro Ribeiro Polla wrote:
>
>   
>> Do you prefer this patch?
>>     
>
> No, that's not even standard C.  The first patch, the one with the
> do { ... } while(0) is good and should be applied.
>
>   

Done.

>> It looked uglier to me, so that's why I preferred fixing pcm.c.
>> But with your explanation, it sounds saner.
>>
>> After intreadwrite.h gets fixed, should I revert the temporary fix I
>> made on pcm.c (which wasn't the best solution)?
>>     
>
> Yes, please.  I don't like the way it looks now.
>
>   

Done.

Sorry for the mess, and thank you for your patience.

Ramiro Polla




More information about the ffmpeg-devel mailing list