On Thu, Dec 17, 2015 at 2:46 PM, Hendrik Leppkes <h.lepp...@gmail.com> wrote: > On Thu, Dec 17, 2015 at 2:06 PM, Michael Niedermayer <michae...@gmx.at> wrote: >> From: Michael Niedermayer <mich...@niedermayer.cc> >> >> This fixes a regression of the sample from Ticket 2371 >> >> This reverts commit bc66451e5e903698ee0500faf04c1214f3dd157f, reversing >> changes made to 9d1fb9ef313e0fb709ac4c35c7bf00264963fd85. >> --- >> libavcodec/h264.h | 8 +++++++ >> libavcodec/h264_refs.c | 57 >> ++++++++++++++++++++++++++--------------------- >> libavcodec/h264_slice.c | 17 +++++++++++++- >> 3 files changed, 56 insertions(+), 26 deletions(-) > > Reverting big commits to fix one sample and making future merges > harder or impossible seems not like the best option, unless it can be > shown that the entire change is flawed, and not just a small bug. > Otherwise, it should be attempted to fix regressions the usual way - > by fixing things, not reverting. > > Once reverted, the change will never come back, we all know that. > > At least thats IMHO. You are the maintainer of the H264 decoder, so if > you want to revert it, feel free.
Libav apparently has a fix for this one: https://lists.libav.org/pipermail/libav-devel/2015-December/073673.html - Hendrik _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
