On Wed, Oct 24, 2012 at 04:13:46PM +0200, Anton Khirnov wrote:
> 
> On Wed, 24 Oct 2012 11:47:24 +0200, Diego Biurrun <di...@biurrun.de> wrote:
> > On Thu, Aug 09, 2012 at 02:38:18AM +0200, Diego Biurrun wrote:
> > > On Thu, Feb 23, 2012 at 08:28:35PM +0100, Anton Khirnov wrote:
> > > > Module: libav
> > > > Branch: master
> > > > Commit: 4da6d194e5b00404f4d545adcaa8e206592ae746
> > > > 
> > > > Author:    Anton Khirnov <an...@khirnov.net>
> > > > Committer: Anton Khirnov <an...@khirnov.net>
> > > > Date:      Thu Feb 23 08:19:13 2012 +0100
> > > > 
> > > > libxvid: switch to encode2().
> > > > 
> > > > --- a/libavcodec/libxvidff.c
> > > > +++ b/libavcodec/libxvidff.c
> > > > @@ -551,16 +569,16 @@ static av_cold int 
> > > > xvid_encode_close(AVCodecContext *avctx) {
> > > >   * @return Returns new length of frame data
> > > >   */
> > > >  int xvid_strip_vol_header(AVCodecContext *avctx,
> > > > -                  unsigned char *frame,
> > > > +                  AVPacket *pkt,
> > > >                    unsigned int header_len,
> > > >                    unsigned int frame_len) {
> > > 
> > > ditto
> > 
> > .. this one was not.
> 
> I don't see what is to be "fixed" here.

The Doxygen should be adapted to document pkt instead of frame.

> Unlike the stuff I removed, this comment
> is nontrivial and should be kept.

Yes.

Diego
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to