On Mon, Apr 17, 2017 at 5:48 PM, Michael Niedermayer <mich...@niedermayer.cc
> wrote:
> On Sun, Apr 16, 2017 at 10:01:01PM -0700, Brett Harrison wrote:
> > Any comments on this patch?
> >
> > -- Forwarded message --
> > From: Brett Harrison <b
Any comments on this patch?
-- Forwarded message --
From: Brett Harrison <brett.harri...@zyamusic.com>
Date: Tue, Apr 11, 2017 at 1:37 PM
Subject: Re: [FFmpeg-devel] [PATCH] added expr evaluation to drawtext -
fontsize
To: FFmpeg development discussions and patches <ffm
Pinging for comments / review...
On Tue, Apr 4, 2017 at 3:45 PM, Brett Harrison <brett.harri...@zyamusic.com>
wrote:
> Resurrecting this patch.
>
> On Thu, Sep 15, 2016 at 3:20 AM, Michael Niedermayer <
> mich...@niedermayer.cc> wrote:
>
>> On Fri, Sep 09
Resurrecting this patch.
On Thu, Sep 15, 2016 at 3:20 AM, Michael Niedermayer <mich...@niedermayer.cc
> wrote:
> On Fri, Sep 09, 2016 at 05:26:03PM -0700, Brett Harrison wrote:
> > Here are the changes requested
> [...]
> > +static av_cold int parse_fonts
Any feedback on the most recent patch?
On Fri, Sep 9, 2016 at 5:26 PM, Brett Harrison <brett.harri...@zyamusic.com>
wrote:
> Here are the changes requested
>
> On Thu, Sep 8, 2016 at 8:50 AM, Michael Niedermayer <
> mich...@niedermayer.cc> wrote:
>
>> On Tue,
Here are the changes requested
On Thu, Sep 8, 2016 at 8:50 AM, Michael Niedermayer <mich...@niedermayer.cc>
wrote:
> On Tue, Sep 06, 2016 at 10:27:24AM -0700, Brett Harrison wrote:
> > This patch addresses your concerns.
> >
> > On Fri, Sep 2, 2016 at 5:05 PM,
This patch addresses your concerns.
On Fri, Sep 2, 2016 at 5:05 PM, Michael Niedermayer <mich...@niedermayer.cc>
wrote:
> On Fri, Sep 02, 2016 at 03:31:21PM -0700, Brett Harrison wrote:
> > Addressed the following concerns.
> >
> > ===
> >
> >
Addressed the following concerns.
===
>libavfilter/vf_drawtext.c: In function ‘update_fontsize’:
>libavfilter/vf_drawtext.c:422:5: warning: ISO C90 forbids mixed declarations
>and code [->Wdeclaration-after-statement]
Fixed this.
>also patch breaks:
>./ffmpeg -i m.mpg -vf
Most recent patch. I was evaluating fontsize too early before when using
the 'n' variable in equations.
On Thu, Sep 1, 2016 at 3:44 PM, Brett Harrison <brett.harri...@zyamusic.com>
wrote:
> Any feedback on this newest patch?
>
> On Tue, Aug 30, 2016 at 2:17 PM, Brett Harrison
Any feedback on this newest patch?
On Tue, Aug 30, 2016 at 2:17 PM, Brett Harrison <brett.harri...@zyamusic.com
> wrote:
> Since there are differing opinions on how the default fontsize should be
> established this patch adds my changes while preserving the current
> behavio
Since there are differing opinions on how the default fontsize should be
established this patch adds my changes while preserving the current
behavior when fontsize is not specified.
On Tue, Aug 30, 2016 at 2:43 AM, Nicolas George <geo...@nsup.org> wrote:
> Le tridi 13 fructidor, an CCXX
rmayer.cc
> wrote:
> On Sat, Aug 27, 2016 at 04:30:05PM -0700, Brett Harrison wrote:
> > Fixed patch based on comments.
> >
> > This time attaching patch file.
> >
> > On Sat, Aug 27, 2016 at 6:21 AM, Moritz Barsnick <barsn...@gmx.net>
> wrote:
> >
Fixed patch based on comments.
This time attaching patch file.
On Sat, Aug 27, 2016 at 6:21 AM, Moritz Barsnick <barsn...@gmx.net> wrote:
> On Fri, Aug 26, 2016 at 14:37:42 -0700, Brett Harrison wrote:
>
> > +if (diff != 0) {
> > + return diff >
Allows expr evaluation in the fontsize parameter for drawtext.
examples (fontsize 1/3 of video height):
ffmpeg -i http://i.giphy.com/kwAi4WrChkSfm.gif -vf
"drawtext=fontfile=/Library/Fonts/Verdana
Bold.ttf:text='HI':fontcolor=yellow:x=(w-tw)/2:y=(h-th)/1.3:fontsize=h/3"
out.gif
ffmpeg -i
14 matches
Mail list logo