On Sun, 17 Jun 2018 15:40:19 +0300 Shlomi Fish <shlo...@shlomifish.org> wrote:
> On Sun, 17 Jun 2018 03:05:27 +0200 > Michael Niedermayer <mich...@niedermayer.cc> wrote: > > > On Tue, Jun 12, 2018 at 12:53:20PM +0300, Shlomi Fish wrote: > > > This message did not arrive to the list after three submissions. > > > > > > Begin forwarded message: > > > > > > Date: Tue, 12 Jun 2018 12:42:52 +0300 > > > From: Shlomi Fish <shlo...@shlomifish.org> > > > To: ffmpeg-devel@ffmpeg.org > > > Cc: Shlomi Fish <shlo...@shlomifish.org> > > > Subject: [PATCH] Refactor two near-identical clauses. > > > > > > > > > Placed under the Expat licence . All tests pass. > > > --- > > > libavfilter/vf_weave.c | 33 ++++++++++++++------------------- > > > 1 file changed, 14 insertions(+), 19 deletions(-) > > > > > > diff --git a/libavfilter/vf_weave.c b/libavfilter/vf_weave.c > > > index 037f5d1cf2..be371201e1 100644 > > > --- a/libavfilter/vf_weave.c > > > +++ b/libavfilter/vf_weave.c > > > @@ -23,6 +23,7 @@ > > > #include "libavutil/pixdesc.h" > > > #include "avfilter.h" > > > #include "internal.h" > > > +#include <stdbool.h> > > > > > > typedef struct WeaveContext { > > > const AVClass *class; > > > @@ -84,6 +85,8 @@ static int filter_frame(AVFilterLink *inlink, AVFrame > > > *in) AVFilterLink *outlink = ctx->outputs[0]; > > > AVFrame *out; > > > int i; > > > + bool weave; > > > + int field1, field2; > > > > > > if (!s->prev) { > > > s->prev = in; > > > @@ -98,26 +101,18 @@ static int filter_frame(AVFilterLink *inlink, AVFrame > > > *in) } > > > av_frame_copy_props(out, in); > > > > > > + weave = (s->double_weave && !(inlink->frame_count_out & 1)); > > > + field1 = s->first_field * weave; > > > + field2 = s->first_field * !weave; > > > for (i = 0; i < s->nb_planes; i++) { > > > - if (s->double_weave && !(inlink->frame_count_out & 1)) { > > > - av_image_copy_plane(out->data[i] + out->linesize[i] * > > > s->first_field, > > > > this seems to be corrupted by line breaks > > > > Well, the git send-email email was silently dropped three times... See: > > http://www.shlomifish.org/Files/files/code/0001-Refactor-two-near-identical-clauses.patch > > also attached here. Email has sadly become unreliable. > Ping! Please review. > > [...] > > > > -- ----------------------------------------------------------------- Shlomi Fish http://www.shlomifish.org/ UNIX Fortune Cookies - http://www.shlomifish.org/humour/fortunes/ Every successful open source project will eventually spawn a sub‐project. — http://www.shlomifish.org/humour/fortunes/osp_rules.html Please reply to list if it's a mailing list post - http://shlom.in/reply . _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel