Re: [FFmpeg-devel] [PATCH] avcodec/flacenc: Invalid Rice order

2015-06-16 Thread George Boyle
On 16/06/15 12:02, Michael Niedermayer wrote: > > please post a patch that contains the remaining 2 hunks in >24h > > also for future fate tests, small samples are preferred to keep the > fate testsuite manageable size wise Ok, will do. Thanks George _

Re: [FFmpeg-devel] [PATCH] avcodec/flacenc: Invalid Rice order

2015-06-16 Thread Michael Niedermayer
On Tue, Jun 16, 2015 at 08:25:01AM +0100, George Boyle wrote: > Fixes ticket #4628. > > The problem arose, in the sample file at least, in the last block where the > minimum and maximum Rice partition orders were both 0. In that case, and any > other where pmax == pmin, the original UINT32_MAX pla

[FFmpeg-devel] [PATCH] avcodec/flacenc: Invalid Rice order

2015-06-16 Thread George Boyle
Fixes ticket #4628. The problem arose, in the sample file at least, in the last block where the minimum and maximum Rice partition orders were both 0. In that case, and any other where pmax == pmin, the original UINT32_MAX placeholder value for bits[opt_porder] was getting overwritten before the c