Re: [PATCH] fix broken build since r38235
Reimar Döffinger <[email protected]> Sun, 24 Jan 2021 13:44:54 +0100
| Newsgroups | gmane.comp.video.mplayer.devel |
|---|---|
| Message-ID | <YA1rxgV6XDeaL/[email protected]> |
On Sat, Jan 23, 2021 at 11:56:34PM +0100, Erik Auerswald wrote: > Hi, > > On 23.01.21 23:28, Reimar Döffinger wrote: > > > 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. > > Thanks, I did not know that (and did not investigate, my fault). That's fine, we in fact had some code using DECLARE_ALIGNED for stack variables and in practice it seems to have worked anyway. So your solution was not entirely wrong, just not quite right. _______________________________________________ MPlayer-dev-eng mailing list [email protected] https://lists.mplayerhq.hu/mailman/listinfo/mplayer-dev-eng