[FFmpeg-devel] Add 64-bit bytestream functions

Måns Rullgård mans
Sat Jul 7 23:51:59 CEST 2007


Michael Niedermayer <michaelni at gmx.at> writes:

> Hi
>
> On Sat, Jul 07, 2007 at 08:26:49PM +0100, M?ns Rullg?rd wrote:
>> Michael Niedermayer <michaelni at gmx.at> writes:
>> 
>> > Hi
>> >
>> > On Sat, Jul 07, 2007 at 04:22:40PM +0100, Mans Rullgard wrote:
>> >> 
>> >> The following patches add 64-bit bytestream functions and fix
>> >> whitespace cosmetics.
>> >> 
>> >> OK to apply?
>> >
>> > are they needed for anything?
>> >
>> > if not they are just unused functions bloating up the code ...
>> 
>> I was doing some cleaning of oggparseogm.c and could use them instead
>> of AV_RL64(buf); buf+=8.  That was the intention with bytestream_*,
>> right?
>
> of course
> if they are needed then the patch is ok ...

Committed and made use of.

> i just didnt want to fill the file with unused functions ...

I don't generally do things without some kind of reason.

-- 
M?ns Rullg?rd
mans at mansr.com




More information about the ffmpeg-devel mailing list