[FFmpeg-devel] [PATCH] avfilter/vf_tile: remove limit of max tile size

Paul B Mahol onemda at gmail.com
Sat Oct 28 17:41:56 EEST 2017


On 10/28/17, Paul B Mahol <onemda at gmail.com> wrote:
> On 10/28/17, Clement Boesch <u at pkh.me> wrote:
>> On Sat, Oct 28, 2017 at 02:42:26PM +0200, Paul B Mahol wrote:
>>> On 10/28/17, Clement Boesch <u at pkh.me> wrote:
>>> > On Fri, Oct 27, 2017 at 10:38:11PM +0200, Nicolas George wrote:
>>> >> Le sextidi 6 brumaire, an CCXXVI, Paul B Mahol a ecrit :
>>> >> > Signed-off-by: Paul B Mahol <onemda at gmail.com>
>>> >> > ---
>>> >> >  libavfilter/vf_tile.c | 8 --------
>>> >> >  1 file changed, 8 deletions(-)
>>> >>
>>> >> Nack.
>>> >>
>>> >> ./ffmpeg_g -lavfi testsrc2=s=2x2,tile=65536x65536,scale=1024x1024 -f
>>> >> framecrc -
>>> >>
>>> >> hangs instead of returning a proper error.
>>> >>
>>> >
>>> > In case there is confusion with what Paul said: `ffmpeg -lavfi
>>> > testsrc2=s=2x2 -f framecrc -` hangs with current git/master, so the
>>> > hang
>>> > you're reporting is likely unrelated to this patch. Instead, it looks
>>> > like
>>> > a
>>> > problem related to the 2x2 size being too small for testsrc2 (testsrc
>>> > supports 2x2, testsrc2 doesn't).
>>> >
>>> > (Not saying the current patch is correct though)
>>>
>>> I don't remmember I hired lawyer. Did I?
>>
>> I'm not defending you, I'm explaining a potential misunderstanding
>> between
>> Nicolas and you, giving information you should probably have provided
>> yourself to ease the communication to prevent pointless escalation.
>
> I already provided it. The problem is people forgot to read.
>

Gonna apply this patch soonTM because of no valid reasons to block it.


More information about the ffmpeg-devel mailing list