[Ffmpeg-devel] [PATCH] bmp decoder using bytestream

Panagiotis Issaris takis
Wed Jan 31 03:00:18 CET 2007


Hi Michel,

Michel Bardiaux wrote:
> Michael Niedermayer wrote:
>> Hi
>>
>> On Tue, Jan 23, 2007 at 03:33:52PM +0100, Alex Beregszaszi wrote:
>>> > Hi,
>>> >
>>>> > > +#define read16(bits) bswap_16(get_bits(bits, 16))
>>>> > > +#define read32(bits) bswap_32(get_bits_long(bits, 32))
>>> > ...
>>>> > > +#define write16(bits, val) put_bits(bits, 16, bswap_16(val))
>>>> > > +#define write32(bits, val) put_bits(bits, 32, bswap_32(val))
>>> > > What about using bytestream.h or intreadwrite.h instead this? I 
>>> dont see
>>> > its putting/reading non-8bit aligned data:
>>
>> no objections feel free to send a patch after this has been applied
>
> Here it is.
> [...]
Applied.

Thanks for the patch!

With friendly regards,
Takis





More information about the ffmpeg-devel mailing list