[MPlayer-dev-eng] [PATCH] VF Overlay

Benjamin Zores ben at geexbox.org
Fri Aug 14 08:56:02 CEST 2009


On Sat, Aug 8, 2009 at 6:08 PM, Reimar
Döffinger<Reimar.Doeffinger at gmx.de> wrote:
> On Sat, Aug 08, 2009 at 04:57:12PM +0200, Benjamin Zores wrote:
>> Jason Tackaberry wrote:
>> > When you get a chance, have a look at the AV_WN32 macro as Reimar
>> > suggested.
>> >
>> > I guess it would look like:
>> >
>> >    AV_WN32(p_bgr24, p_bgr32);
>> >    *p_alpha = p_bgr32[3];
>>
>> Actually it was:
>> AV_WN32(p_bgr24, AV_RN32(p_bgr32));
>
> Well, p_bgr32 is aligned though, so on some hardware this would be
> faster:
> AV_WN32(p_bgr24, *(uint32_t *)p_bgr32);

Indeed, this works pretty fine.
Unless further comments, I plan on applying this patch by tonight/this weekend.

Ben

-- 
"My life, and by extension everyone else's is meaningless."
Bender, Futurama



More information about the MPlayer-dev-eng mailing list