Re: [FFmpeg-devel] [PATCH 1/2] avfilter: vf_minterpolate: rename chroma log vars

2016-09-02 Thread Paul B Mahol
On 8/30/16, Davinder Singh wrote: > On Mon, Aug 29, 2016 at 8:59 PM Paul B Mahol wrote: > [...] > > And using avcodec* stuff is in lavfi is not needed. > > uses AVPixFmtDesc now. should I also not use libavcodec/mathops.h? median > of 3 function is shared.

Re: [FFmpeg-devel] [PATCH 1/2] avfilter: vf_minterpolate: rename chroma log vars

2016-08-29 Thread Paul B Mahol
On Tuesday, August 30, 2016, Davinder Singh wrote: > On Mon, Aug 29, 2016 at 8:59 PM Paul B Mahol > wrote: > [...] > > And using avcodec* stuff is in lavfi is not needed. > > uses AVPixFmtDesc now. should I also not use libavcodec/mathops.h?

Re: [FFmpeg-devel] [PATCH 1/2] avfilter: vf_minterpolate: rename chroma log vars

2016-08-29 Thread Davinder Singh
On Mon, Aug 29, 2016 at 8:59 PM Paul B Mahol wrote: [...] And using avcodec* stuff is in lavfi is not needed. uses AVPixFmtDesc now. should I also not use libavcodec/mathops.h? median of 3 function is shared. can it be moved? updated patch attached.

Re: [FFmpeg-devel] [PATCH 1/2] avfilter: vf_minterpolate: rename chroma log vars

2016-08-28 Thread Paul B Mahol
On Sun, Aug 28, 2016 at 10:47 PM, Michael Niedermayer < mich...@niedermayer.cc> wrote: > On Sun, Aug 28, 2016 at 06:24:14PM +, Davinder Singh wrote: > > hi, > > > > this rename confusing chroma variables to one used in AVPixFmtDescriptor. > > more consistent. > > also removed some useless

Re: [FFmpeg-devel] [PATCH 1/2] avfilter: vf_minterpolate: rename chroma log vars

2016-08-28 Thread Michael Niedermayer
On Sun, Aug 28, 2016 at 06:24:14PM +, Davinder Singh wrote: > hi, > > this rename confusing chroma variables to one used in AVPixFmtDescriptor. > more consistent. > also removed some useless vars from context. > > thanks > DSM_ [...] > @@ -900,6 +894,8 @@ static void

[FFmpeg-devel] [PATCH 1/2] avfilter: vf_minterpolate: rename chroma log vars

2016-08-28 Thread Davinder Singh
hi, this rename confusing chroma variables to one used in AVPixFmtDescriptor. more consistent. also removed some useless vars from context. thanks DSM_ 0001-avfilter-vf_minterpolate-rename-chroma-log-vars.patch Description: Binary data ___