On Tue, Apr 05, 2016 at 10:45:18PM +0200, Michael Niedermayer wrote:
> Hi Disha
>
> On Tue, Apr 05, 2016 at 06:50:33AM +0530, Disha Singh wrote:
> > This patch has an lpc issue, and needs some work with passing of parameters
> > in ff_lpc_calc_coeff(), which my mentor said he would help with.
> [...]
>
> > +static ChannelParams restart_channel_params[MAX_CHANNELS];
> > +static DecodingParams restart_decoding_params[MAX_SUBSTREAMS];
> > +static BestOffset restart_best_offset[NUM_CODEBOOKS] = {{0}};
>
> "non const" statics are not allowed as they break with muliple
> encoder instancesthat is if they would change, if they do not change they should be const [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB If a bugfix only changes things apparently unrelated to the bug with no further explanation, that is a good sign that the bugfix is wrong.
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
