Re: [FFmpeg-devel] [PATCH 1/2] wmavoice: don't check if we have enough bits to read a superframe.

2016-12-16 Thread Michael Niedermayer
On Fri, Dec 16, 2016 at 09:51:07AM -0500, Ronald S. Bultje wrote: > Hi, > > On Fri, Dec 16, 2016 at 9:49 AM, Michael Niedermayer > wrote: > > > On Fri, Dec 16, 2016 at 08:19:44AM -0500, Ronald S. Bultje wrote: > > > The checked bitstream reader makes this unnecessary. >

Re: [FFmpeg-devel] [PATCH 1/2] wmavoice: don't check if we have enough bits to read a superframe.

2016-12-16 Thread Ronald S. Bultje
Hi, On Fri, Dec 16, 2016 at 9:49 AM, Michael Niedermayer wrote: > On Fri, Dec 16, 2016 at 08:19:44AM -0500, Ronald S. Bultje wrote: > > The checked bitstream reader makes this unnecessary. > > --- > > libavcodec/wmavoice.c | 83 -- >

Re: [FFmpeg-devel] [PATCH 1/2] wmavoice: don't check if we have enough bits to read a superframe.

2016-12-16 Thread Michael Niedermayer
On Fri, Dec 16, 2016 at 08:19:44AM -0500, Ronald S. Bultje wrote: > The checked bitstream reader makes this unnecessary. > --- > libavcodec/wmavoice.c | 83 > --- > 1 file changed, 83 deletions(-) this breaks fate TESTwmavoice-19k

Re: [FFmpeg-devel] [PATCH 1/2] wmavoice: don't check if we have enough bits to read a superframe.

2016-12-16 Thread Paul B Mahol
On 12/16/16, Ronald S. Bultje wrote: > The checked bitstream reader makes this unnecessary. > --- > libavcodec/wmavoice.c | 83 > --- > 1 file changed, 83 deletions(-) > lgtm ___

[FFmpeg-devel] [PATCH 1/2] wmavoice: don't check if we have enough bits to read a superframe.

2016-12-16 Thread Ronald S. Bultje
The checked bitstream reader makes this unnecessary. --- libavcodec/wmavoice.c | 83 --- 1 file changed, 83 deletions(-) diff --git a/libavcodec/wmavoice.c b/libavcodec/wmavoice.c index ceac61f..0f29bdd 100644 --- a/libavcodec/wmavoice.c +++