Re: [FFmpeg-devel] [PATCH 1/2] wmavoice: truncate spillover_nbits if too large

2017-01-02 Thread Andreas Cadhalpun
On 02.01.2017 04:14, Ronald S. Bultje wrote: > On Sun, Jan 1, 2017 at 5:55 PM, Andreas Cadhalpun > wrote: > So what would you do instead? > > I'd just remove the message, but otherwise what you're doing (truncate > spillover_nbits) seems fine. OK. While I think that the message could have

Re: [FFmpeg-devel] [PATCH 1/2] wmavoice: truncate spillover_nbits if too large

2017-01-01 Thread Ronald S. Bultje
Hi, On Sun, Jan 1, 2017 at 5:55 PM, Andreas Cadhalpun < andreas.cadhal...@googlemail.com> wrote: > On 01.01.2017 23:27, Ronald S. Bultje wrote: > > On Sun, Jan 1, 2017 at 5:18 PM, Andreas Cadhalpun < > andreas.cadhal...@googlemail.com > > wrote: > > > >

Re: [FFmpeg-devel] [PATCH 1/2] wmavoice: truncate spillover_nbits if too large

2017-01-01 Thread Andreas Cadhalpun
On 01.01.2017 23:27, Ronald S. Bultje wrote: > On Sun, Jan 1, 2017 at 5:18 PM, Andreas Cadhalpun > mailto:andreas.cadhal...@googlemail.com>> > wrote: > > This fixes triggering the av_assert0(ret <= tmp.size). > > The problem was reintroduced by commit > 7b27dd5c16de785297ce4de4b88af

Re: [FFmpeg-devel] [PATCH 1/2] wmavoice: truncate spillover_nbits if too large

2017-01-01 Thread Ronald S. Bultje
Hi, On Sun, Jan 1, 2017 at 5:18 PM, Andreas Cadhalpun < andreas.cadhal...@googlemail.com> wrote: > This fixes triggering the av_assert0(ret <= tmp.size). > > The problem was reintroduced by commit > 7b27dd5c16de785297ce4de4b88afa0b6685f61d and originally fixed by > 2a4700a4f03280fa8ba4fc0f8a9987b

[FFmpeg-devel] [PATCH 1/2] wmavoice: truncate spillover_nbits if too large

2017-01-01 Thread Andreas Cadhalpun
This fixes triggering the av_assert0(ret <= tmp.size). The problem was reintroduced by commit 7b27dd5c16de785297ce4de4b88afa0b6685f61d and originally fixed by 2a4700a4f03280fa8ba4fc0f8a9987bb550f0d1e. Signed-off-by: Andreas Cadhalpun --- libavcodec/wmavoice.c | 5 + 1 file changed, 5 insert