Re: [FFmpeg-devel] [PATCH] avfilter/drawbox: rename variable for maximum thickness

2017-11-28 Thread Michael Niedermayer
On Mon, Nov 27, 2017 at 10:10:08AM +0530, Gyan Doshi wrote: > > On 11/23/2017 7:12 PM, Gyan Doshi wrote: > > > >On 11/20/2017 3:59 PM, Gyan Doshi wrote: > >>At present, the value name 'max' for maximum thickness in > >>drawbox (and drawgrid) filter leads to a parse error if the > >>thickness expre

Re: [FFmpeg-devel] [PATCH] avfilter/drawbox: rename variable for maximum thickness

2017-11-26 Thread Gyan Doshi
On 11/23/2017 7:12 PM, Gyan Doshi wrote: On 11/20/2017 3:59 PM, Gyan Doshi wrote: At present, the value name 'max' for maximum thickness in drawbox (and drawgrid) filter leads to a parse error if the thickness expression contains 'max(val1,val2)' i.e. [Eval @ ...] Invalid chars '(20,30

Re: [FFmpeg-devel] [PATCH] avfilter/drawbox: rename variable for maximum thickness

2017-11-23 Thread Gyan Doshi
On 11/20/2017 3:59 PM, Gyan Doshi wrote: At present, the value name 'max' for maximum thickness in drawbox (and drawgrid) filter leads to a parse error if the thickness expression contains 'max(val1,val2)' i.e.     [Eval @ ...] Invalid chars '(20,30)' at the end of expression 'max(20,30)'