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

[Hendrik Leppkes]

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.

- Hendrik