[FFmpeg-devel] [PATCH v3] avfilter/scale: refactor common code for scaling height/width expressions

Mark Thompson sw at jkqxz.net
Fri Feb 3 01:02:51 EET 2017


On 02/02/17 12:27, Michael Niedermayer wrote:
> On Thu, Feb 02, 2017 at 12:35:45AM +0000, Mark Thompson wrote:
>> On 02/02/17 00:30, Aman Gupta wrote:
>>> From: Aman Gupta <aman at tmm1.net>
>>>
>>> Implements support for height/width expressions in vf_scale_vaapi,
>>> by refactoring common code into a new libavfilter/scale.c
>>> ---
>>>  libavfilter/Makefile         |   8 +--
>>>  libavfilter/scale.c          | 152 +++++++++++++++++++++++++++++++++++++++++++
>>>  libavfilter/scale.h          |  28 ++++++++
>>>  libavfilter/vf_scale.c       | 109 +++----------------------------
>>>  libavfilter/vf_scale_npp.c   |  93 +++-----------------------
>>>  libavfilter/vf_scale_vaapi.c |  19 ++++--
>>>  6 files changed, 216 insertions(+), 193 deletions(-)
>>>  create mode 100644 libavfilter/scale.c
>>>  create mode 100644 libavfilter/scale.h
>>
>> LGTM.  Michael is listed as the maintainer of vf_scale, though, so we should wait for him to look at it before applying.
> 
> assuming the code is just moved/factored, sure LGTM

Applied.

Thanks,

- Mark


More information about the ffmpeg-devel mailing list