[FFmpeg-devel] [PATCH] dsputil: add bswap16_buf()

Måns Rullgård mans
Thu Jun 17 12:43:44 CEST 2010


Michael Niedermayer <michaelni at gmx.at> writes:

> On Thu, Jun 17, 2010 at 11:24:11AM +0100, M?ns Rullg?rd wrote:
>> Mans Rullgard <mans at mansr.com> writes:
>> 
>> > ---
>> >  libavcodec/dsputil.c |    7 +++++++
>> >  libavcodec/dsputil.h |    1 +
>> >  2 files changed, 8 insertions(+), 0 deletions(-)
>> 
>> Is anyone fundamentally opposed to this?  If not, can we please just
>> apply it and figure out the required alignment later?
>
> without the alignment being documented its not possible to implement it
> efficiently, thus useless

No more useless than having a dozen copies of that loop scattered
about the code.

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



More information about the ffmpeg-devel mailing list