Re: [FFmpeg-devel] [PATCH 1/2] avcodec/cbs_h265: fix valid range of num_tile_{columns, rows}_minus1 in H265RawPPS

2019-06-21 Thread James Almer
On 6/21/2019 1:24 AM, Gyan wrote: > > > On 20-06-2019 11:15 PM, James Almer wrote: >> The spec states they can't be higher than the respective dimensions of >> the >> stream in CTBs. >> >> Signed-off-by: James Almer >> --- >> I don't think it's wise further limiting the range to the maximum >> c

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/cbs_h265: fix valid range of num_tile_{columns, rows}_minus1 in H265RawPPS

2019-06-20 Thread Gyan
On 20-06-2019 11:15 PM, James Almer wrote: The spec states they can't be higher than the respective dimensions of the stream in CTBs. Signed-off-by: James Almer --- I don't think it's wise further limiting the range to the maximum currently defined for level 6.2 using those two HEVC_ defines,

[FFmpeg-devel] [PATCH 1/2] avcodec/cbs_h265: fix valid range of num_tile_{columns, rows}_minus1 in H265RawPPS

2019-06-20 Thread James Almer
The spec states they can't be higher than the respective dimensions of the stream in CTBs. Signed-off-by: James Almer --- I don't think it's wise further limiting the range to the maximum currently defined for level 6.2 using those two HEVC_ defines, since a stream could in theory go beyond them