[FFmpeg-devel] [PATCH] libavcodec/mmaldec.c: add interlaced_frame and format struct to AVFrame for deinterlacing
Jens Ziller
zillevdr at gmx.de
Sat Jul 16 11:08:12 CEST 2016
Am Sonntag, den 03.07.2016, 19:36 +0200 schrieb Jens Ziller:
> Am Sonntag, den 03.07.2016, 18:05 +0200 schrieb Moritz Barsnick:
> >
> > On Sun, Jul 03, 2016 at 17:20:41 +0200, Jens Ziller wrote:
> > >
> > >
> > > Am Samstag, den 02.07.2016, 17:54 +0200 schrieb Moritz Barsnick:
> > > >
> > > >
> > > > On Sun, Jun 26, 2016 at 17:12:14 +0200, Jens Ziller wrote:
> > > > >
> > > > >
> > > > >
> > > > > + ctx->interlaced_frame = !(interlace_type.eMode ==
> > > > > MMAL_InterlaceProgressive);
> > > > What's wrong with using the "!=" operator instead?
> > > "!=" is a comparing. "= !()" assign with a negate. Here is "=
> > > !()"
> > > needed.
> > I meant the comparison, not the assignment, so replacing:
> > ctx->interlaced_frame = !(interlace_type.eMode ==
> > MMAL_InterlaceProgressive)
> > with
> > ctx->interlaced_frame = (interlace_type.eMode !=
> > MMAL_InterlaceProgressive)
> >
> > The former is rather ... convoluted.
> > (Brackets optional, but probably better for readability.)
> >
> > Moritz
> Oh, sorry! I'am so blind. Yes, that's not really smart. I changed
> that.
> The new patch is attached.
>
> Regards Jens
>
Hello Michael and list,
this patch was discussed and improved on the ML. What can I still do
that my app get interlaced_frame and a pointer to the existing
MMAL_ES_FORMAT_T from ffmpeg? mmaldec.c have unfortunately no
maintainer.
regards Jens
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-v4-fill-AVFrame-interlaced_frame-with-MMAL_PARAMETER.patch
Type: text/x-patch
Size: 3113 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160716/c646233e/attachment.bin>
More information about the ffmpeg-devel
mailing list