[FFmpeg-devel] [FFmpeg-cvslog] Merge commit '4fb311c804098d78e5ce5f527f9a9c37536d3a08'

Michael Niedermayer michael at niedermayer.cc
Mon Mar 20 13:29:49 EET 2017


On Mon, Mar 20, 2017 at 10:14:21AM +0100, Hendrik Leppkes wrote:
> On Mon, Mar 20, 2017 at 10:11 AM, Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:
> > 2017-03-20 10:03 GMT+01:00 Hendrik Leppkes <h.leppkes at gmail.com>:
> >> On Mon, Mar 20, 2017 at 9:59 AM, Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:
> >>> 2017-03-20 9:42 GMT+01:00 Clément Bœsch <u at pkh.me>:
> >>>> On Mon, Mar 20, 2017 at 09:38:15AM +0100, Carl Eugen Hoyos wrote:
> >>>>> 2017-03-20 9:35 GMT+01:00 Clément Bœsch <u at pkh.me>:
> >>>>> > On Mon, Mar 20, 2017 at 09:17:36AM +0100, Carl Eugen Hoyos wrote:
> >>>>> >> 2017-03-20 8:57 GMT+01:00 Clément Bœsch <git at videolan.org>:
> >>>>> >> > ffmpeg | branch: master | Clément Bœsch <u at pkh.me> | Mon Mar 20 08:52:07 2017 +0100| [3835283293bfd38ba69203f4618f0f0f21377bcc] | committer: Clément Bœsch
> >>>>> >> >
> >>>>> >> > Merge commit '4fb311c804098d78e5ce5f527f9a9c37536d3a08'
> >>>>> >> >
> >>>>> >> > * commit '4fb311c804098d78e5ce5f527f9a9c37536d3a08':
> >>>>> >> >   Drop memalign hack
> >>>>> >> >
> >>>>> >> > Merged, as this may indeed be uneeded since
> >>>>> >> > 46e3936fb04d06550151e667357065e3f646da1a.
> >>>>> >>
> >>>>> >> How are we supposed to test the next crash because
> >>>>> >> of missing alignment?
> >>>>> >>
> >>>>> >
> >>>>> > I thought it was only about compatibility with platforms not having
> >>>>> > an aligned allocator.
> >>>>>
> >>>>> If you think such platforms exist, one more reason not to merge...
> >>>>
> >>>> There was apparently MinGW but it seemed to have been fixed
> >>>> (see the hash reference in the merge description).
> >>>
> >>> I can only add that the simple idea of removing support for
> >>> platforms without any cause fills me with shame.
> >>>
> >>
> >> If you can name a platfom that needs this in current git master, then
> >> we can move forward.
> >
> > Old MingW comes to mind...
> >
> 
> As was pointed out above, old mingw was fixed in another commit right
> before this one.
> If you can find an actual reproducable case on current git master,
> please do provide said information.

Is the mixture of memalign() and free() spec compliant ?

IIRC you said in IRC memalign and free would be ok in posix but as i
checked now i cannot find memalign in posix only posix_memalign()
but maybe i looked at the wrong place


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The worst form of inequality is to try to make unequal things equal.
-- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170320/1d40f593/attachment.sig>


More information about the ffmpeg-devel mailing list