Re: [FFmpeg-devel] [PATCH v7] Add lensfun filter

2018-07-15 Thread Stephen Seo
Hi, Oh, sorry please ignore my previous email, I just noticed the patch got merged into the repository. Thanks, Stephen Seo On Mon, Jul 16, 2018 at 10:27 AM, Stephen Seo wrote: > On Fri, Jul 13, 2018 at 7:33 PM, Stephen Seo > wrote: > >> v7 of this patch fixes use of sqrt() (c

Re: [FFmpeg-devel] [PATCH v7] Add lensfun filter

2018-07-15 Thread Stephen Seo
On Fri, Jul 13, 2018 at 7:33 PM, Stephen Seo wrote: > v7 of this patch fixes use of sqrt() (changed to sqrtf(); thanks to > Moritz Barsnick for pointing this out). Also should be rebased off of > latest master branch commits at this point. > > Signed-off-by: Stephen Seo >

[FFmpeg-devel] [PATCH v7] Add lensfun filter

2018-07-13 Thread Stephen Seo
-by: Stephen Seo --- configure| 4 + doc/filters.texi | 118 + libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 + libavfilter/vf_lensfun.c | 548 +++ 5 files changed, 672 insertions(+) create mode 100644

Re: [FFmpeg-devel] [PATCH v5] Add lensfun filter

2018-07-12 Thread Stephen Seo
he best match, first in the list returned from the lensfun API). Everything else makes sense though. I will send another patch once I'm done with the fixes. Thanks, Stephen Seo ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH v6] Add lensfun filter

2018-07-13 Thread Stephen Seo
the issue). v6 of this patch fixes more problems (thanks to Moritz Barsnick for pointing them out). Signed-off-by: Stephen Seo --- configure| 4 + doc/filters.texi | 118 + libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 + libavfilter

[FFmpeg-devel] mistake in patch "Add lensfun filter"

2018-07-11 Thread Stephen Seo
? (I'm a bit new to submitting code to mailing lists.) Thanks, Stephen Seo ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] Add lensfun filter

2018-07-11 Thread Stephen Seo
artifacts in the corrected image in my tests. The lanczos interpolation is derived from lenstool's implementation of lanczos interpolation. Lenstool is an app within the lensfun repository which is licensed under GPL3. Signed-off-by: Stephen Seo --- configure| 5 + doc

[FFmpeg-devel] [PATCH v2] Add lensfun filter

2018-07-11 Thread Stephen Seo
-by: Stephen Seo --- configure| 5 + doc/filters.texi | 103 +++ libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 + libavfilter/vf_lensfun.c | 605 +++ 5 files changed, 715 insertions(+) create mode 100644 libavfilter

[FFmpeg-devel] [PATCH v3] Add lensfun filter

2018-07-12 Thread Stephen Seo
of this patch fixes code style and dependency to gplv3 (thanks to Paul B Mahol for pointing out the mentioned issues). Signed-off-by: Stephen Seo --- configure| 5 + doc/filters.texi | 103 libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 + libavfilter

Re: [FFmpeg-devel] mistake in patch "Add lensfun filter"

2018-07-12 Thread Stephen Seo
On Thu, Jul 12, 2018 at 5:52 PM, Timo Rothenpieler wrote: > On 12/07/2018 06:40, Stephen Seo wrote: > >> Hi, >> >> It appears I used the wrong license in libavfilter/vf_lensfun.c >> >> It derives from the Lensfun library that uses (L)GPL3, but I used GPL2 by

[FFmpeg-devel] [PATCH v4] Add lensfun filter

2018-07-12 Thread Stephen Seo
of this patch fixes code style and dependency to gplv3 (thanks to Paul B Mahol for pointing out the mentioned issues). v4 of this patch fixes more code style issues that were missed in v3. Signed-off-by: Stephen Seo --- configure| 5 + doc/filters.texi | 103 libavfilter

[FFmpeg-devel] [PATCH v5] Add lensfun filter

2018-07-12 Thread Stephen Seo
the issue). Signed-off-by: Stephen Seo --- configure| 5 + doc/filters.texi | 122 + libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 + libavfilter/vf_lensfun.c | 543 +++ 5 files changed, 672 insertions

Re: [FFmpeg-devel] mistake in patch "Add lensfun filter"

2018-07-12 Thread Stephen Seo
On Thu, Jul 12, 2018 at 7:10 PM, Gyan Doshi wrote: > > > On 12-07-2018 02:45 PM, Stephen Seo wrote: > > > I have already added some lines in configure to not compile the filter >>> >> unless --enable-version3 is also specified to configure, but thanks for >