[MPlayer-dev-eng] [PATCH] fix broken build since r38235 (was: Re: [MPlayer-cvslog] r38235 - in trunk: libmpcodecs/vf_filmdint.c libmpcodecs/vf_gradfun.c libmpcodecs/vf_ow.c libmpcodecs/vf_spp.c m_config.h sub/osd.c)

Reimar Döffinger Reimar.Doeffinger at gmx.de
Sun Jan 24 00:28:09 EET 2021



> On 23 Jan 2021, at 21:16, Erik Auerswald <auerswal at unix-ag.uni-kl.de> wrote:
> 
> Hi,
> 
> 
> On 23.01.21 19:52, Erik Auerswald wrote:
>> Hi,
>> On 23.01.21 19:22, reimar wrote:
>>> Author: reimar
>>> Date: Sat Jan 23 20:22:15 2021
>>> New Revision: 38235
>> After the recent changes, MPlayer no longer compiles for me.
>> since the problem occurs in the file libmpcodecs/vf_spp.c
>> I assume that this check in to be the cause.
>>> Log:
>>> Switch to using DECLARE_ALIGNED for better portability.
> 
> Using DECLARE_ALIGNED instead of LOCAL_ALIGNED_16 fixes the
> compilation error for me, please see attached patch.  I hope
> that I've used DECLARE_ALIGNED correctly.

Using DECLARE_ALIGNED is not correct to use for stack variables.
I guess I must have had some interaction with one of my other
patches in testing so I didn’t see the failure.
I hope it’s fixed now with the missing include added.
Thanks for testing my stuff, much appreciated!


More information about the MPlayer-dev-eng mailing list