On Mon, Aug 06, 2012 at 03:01:19PM +0200, Janne Grunau wrote:
> On 2012-08-06 14:50:57 +0200, Kostya Shishkov wrote:
> > On Mon, Aug 06, 2012 at 02:48:44PM +0200, Janne Grunau wrote:
> > > ---
> > >  libavcodec/nuv.c |    2 --
> > >  1 files changed, 0 insertions(+), 2 deletions(-)
> > > 
> > > diff --git a/libavcodec/nuv.c b/libavcodec/nuv.c
> > > index 4cb9b95..14e9c79 100644
> > > --- a/libavcodec/nuv.c
> > > +++ b/libavcodec/nuv.c
> > > @@ -254,8 +254,6 @@ static av_cold int decode_init(AVCodecContext *avctx) 
> > > {
> > >      c->pic.data[0] = NULL;
> > >      c->decomp_buf = NULL;
> > >      c->quality = -1;
> > > -    c->width = 0;
> > > -    c->height = 0;
> > >      c->codec_frameheader = avctx->codec_tag == MKTAG('R', 'J', 'P', 'G');
> > >      if (avctx->extradata_size)
> > >          get_quant(avctx, c, avctx->extradata, avctx->extradata_size);
> > > -- 
> > 
> > Why not leave it as it?
> 
> it's unecessary since priv_data is zero-ed out. Idid it only because I touched
> the lines in the cosmetics patch. I won't insist on it though.

Neither shall I on dropping it.
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to