[FFmpeg-devel] [PATCH] add signature filter for MPEG7 video signature

[Gerion Entrup]

On Sonntag, 20. März 2016 15:46:12 CET Gerion Entrup wrote:
> On Sonntag, 20. März 2016 12:57:33 CET Michael Niedermayer wrote:
> > On Sun, Mar 20, 2016 at 12:00:13PM +0100, Gerion Entrup wrote:
> > [...]
> > 
> > > +++ b/libavfilter/signature.h
> > > @@ -0,0 +1,569 @@
> > > +/*
> > > + * Copyright (c) 2016 Gerion Entrup
> > > + *
> > > + * This file is part of FFmpeg.
> > > + *
> > > + * FFmpeg is free software; you can redistribute it and/or modify
> > > + * it under the terms of the GNU General Public License as published by
> > > + * the Free Software Foundation; either version 2 of the License, or
> > > + * (at your option) any later version.
> > > + *
> > > + * FFmpeg is distributed in the hope that it will be useful,
> > > + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> > > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> > > + * GNU General Public License for more details.
> > > + *
> > > + * You should have received a copy of the GNU General Public License
> > > along
> > > + * with FFmpeg; if not, write to the Free Software Foundation, Inc.,
> > > + * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
> > > + */
> > > +
> > > +/**
> > > + * @file
> > > + * MPEG-7 video signature calculation and lookup filter
> > > + */
> > > +
> > > +#include <float.h>
> > > +#include "libavutil/opt.h"
> > > +#include "libavutil/timestamp.h"
> > > +#include "avfilter.h"
> > > +#include "internal.h"
> > > +#include <stdbool.h>
> > > +
> > > +#define ELEMENT_COUNT 10
> > > +#define SIGELEM_SIZE 380
> > > +#define DIFFELEM_SIZE 348 /* SIGELEM_SIZE - elem_a1 - elem_a2 */
> > > +#define COURSE_SIZE 90
> > > +
> > > +typedef struct {
> > > +    int x;
> > > +    int y;
> > > +} Point;
> > > +
> > > +typedef struct {
> > > +    Point up;
> > > +    Point to;
> > > +} Block;
> > 
> > missing standard duplicate inclusion guards
> > 
> > [...]
> 
> Fixed. Also put { in a seperate line for functions.

On Sonntag, 20. März 2016 12:39:53 CET Paul B Mahol wrote:
> If this is supposed to be GPL only, you need to modify other files.

This was my intention. I've added the missing configure line (sorry for double 
mail).

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-add-signature-filter-for-MPEG7-video-signature.patch
Type: text/x-patch
Size: 72275 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160320/b2ff71ef/attachment.bin>