On Thu, Aug 10, 2017 at 01:43:42AM +0000, Davinder Singh wrote:
> On Thu, Aug 10, 2017 at 6:59 AM Michael Niedermayer <mich...@niedermayer.cc>
> wrote:
>
> > On Wed, Aug 09, 2017 at 07:46:30AM +0000, Davinder Singh wrote:
> > > hi,
> > >
> > > this disables unused function amv_encode_picture() when AMV encoder is
> > > disabled (and mjpeg enabled).
> > > silences this warning:
> > > CC libavcodec/mjpegenc.o
> > > libavcodec/mjpegenc.c:351:12: warning: unused function
> > 'amv_encode_picture'
> > > [-Wunused-function]
> > > static int amv_encode_picture(AVCodecContext *avctx, AVPacket *pkt,
> > > ^
> > >
> > > Patch attached.
> > >
> > > Regards.
> > > --
> > > Davinder Singh
> >
> > > mjpegenc.c | 10 +++++-----
> > > 1 file changed, 5 insertions(+), 5 deletions(-)
> > > 0fe583bfdb304ce3d8881e9836cc1983c65e3a90
> > 0001-avcodec-mjpegenc-disable-unused-code-with-AMV.patch
> > > From cadf679bb0ad6d09d451512238e790645262f2f8 Mon Sep 17 00:00:00 2001
> > > From: Davinder Singh <ds.mud...@gmail.com>
> > > Date: Wed, 9 Aug 2017 13:01:07 +0530
> > > Subject: [PATCH] avcodec/mjpegenc: disable unused code with AMV
> > >
> > > disable unused amv_encode_picture() when AMV encoder is not configured.
> > > minor formatting improvement.
> > > ---
> > > libavcodec/mjpegenc.c | 10 +++++-----
> > > 1 file changed, 5 insertions(+), 5 deletions(-)
> > >
> > > diff --git a/libavcodec/mjpegenc.c b/libavcodec/mjpegenc.c
> > > index ee77cde8cb..e6cdaf6376 100644
> > > --- a/libavcodec/mjpegenc.c
> > > +++ b/libavcodec/mjpegenc.c
> > > @@ -39,7 +39,6 @@
> > > #include "mjpeg.h"
> > > #include "mjpegenc.h"
> > >
> > > -
> > > static int alloc_huffman(MpegEncContext *s)
> > > {
> > > MJpegContext *m = s->mjpeg_ctx;
> >
> > please move unrelated cosmetic chages into a seperate patch
> >
> > Patches attached.applied thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Take away the freedom of one citizen and you will be jailed, take away the freedom of all citizens and you will be congratulated by your peers in Parliament.
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
