Re: [FFmpeg-devel] [PATCH] lavfi: remove the duplicate expression evaluation.

2017-01-09 Thread wm4
On Mon, 9 Jan 2017 11:46:01 +0100 Nicolas George wrote: > Le decadi 20 nivôse, an CCXXV, Jun Zhao a écrit : > > I didn't find any case about x depends on y or y depends y. > > Some users have occasionally asked for it, IIRC. > > > If have this dependency relationship , why the code don't hand

Re: [FFmpeg-devel] [PATCH] lavfi: remove the duplicate expression evaluation.

2017-01-09 Thread Nicolas George
Le decadi 20 nivôse, an CCXXV, Jun Zhao a écrit : > I didn't find any case about x depends on y or y depends y. Some users have occasionally asked for it, IIRC. > If have this dependency relationship , why the code don't handle out_w/out_h > in > crop? Ideally, all variables should be able to

Re: [FFmpeg-devel] [PATCH] lavfi: remove the duplicate expression evaluation.

2017-01-08 Thread Jun Zhao
On 2017/1/9 8:21, Mark Thompson wrote: > On 09/01/17 00:12, Jun Zhao wrote: >> From 80ea322e8bd634dbf93291f7a4d78bc87834c0a8 Mon Sep 17 00:00:00 2001 >> From: Jun Zhao >> Date: Sun, 8 Jan 2017 14:26:39 +0800 >> Subject: [PATCH] lavfi: remove the duplicate expression evaluation. >> >> remove the

Re: [FFmpeg-devel] [PATCH] lavfi: remove the duplicate expression evaluation.

2017-01-08 Thread Mark Thompson
On 09/01/17 00:12, Jun Zhao wrote: > From 80ea322e8bd634dbf93291f7a4d78bc87834c0a8 Mon Sep 17 00:00:00 2001 > From: Jun Zhao > Date: Sun, 8 Jan 2017 14:26:39 +0800 > Subject: [PATCH] lavfi: remove the duplicate expression evaluation. > > remove the duplicate expression evaluation in crop/drawtext

[FFmpeg-devel] [PATCH] lavfi: remove the duplicate expression evaluation.

2017-01-08 Thread Jun Zhao
From 80ea322e8bd634dbf93291f7a4d78bc87834c0a8 Mon Sep 17 00:00:00 2001 From: Jun Zhao Date: Sun, 8 Jan 2017 14:26:39 +0800 Subject: [PATCH] lavfi: remove the duplicate expression evaluation. remove the duplicate expression evaluation in crop/drawtext/overlay. Signed-off-by: Jun Zhao --- libavf