Re: [FFmpeg-devel] [PATCH 5/5] dnxhddec: replace some 0s by DNXHD_VARIABLE

2015-10-05 Thread Christophe Gisquet
Hi, 2015-10-05 10:57 GMT+02:00 tim nicholson : > I thought JJ had got hold of a copy of the specs... He was quick to notice the 2014 ones do not cover DNxHR. But the bitstream doesn't seem to depart that much from the old stuff, and ffmpeg's output after this patch series seems adequate. -- Chr

Re: [FFmpeg-devel] [PATCH 5/5] dnxhddec: replace some 0s by DNXHD_VARIABLE

2015-10-05 Thread tim nicholson
On 05/10/15 08:51, Christophe Gisquet wrote: > Hi, > > 2015-10-05 8:44 GMT+02:00 tim nicholson : >> On 02/10/15 20:00, Christophe Gisquet wrote: >>> A series of 0 in a table can be confusing, and the corresponding checks >>> strange, so using a macro instead of that magic is more readable. >>> ---

Re: [FFmpeg-devel] [PATCH 5/5] dnxhddec: replace some 0s by DNXHD_VARIABLE

2015-10-05 Thread Christophe Gisquet
Hi, 2015-10-05 8:44 GMT+02:00 tim nicholson : > On 02/10/15 20:00, Christophe Gisquet wrote: >> A series of 0 in a table can be confusing, and the corresponding checks >> strange, so using a macro instead of that magic is more readable. >> --- >> libavcodec/dnxhddata.c | 10 +- >> libavco

Re: [FFmpeg-devel] [PATCH 5/5] dnxhddec: replace some 0s by DNXHD_VARIABLE

2015-10-04 Thread tim nicholson
On 02/10/15 20:00, Christophe Gisquet wrote: > A series of 0 in a table can be confusing, and the corresponding checks > strange, so using a macro instead of that magic is more readable. > --- > libavcodec/dnxhddata.c | 10 +- > libavcodec/dnxhddata.h | 3 +++ > libavcodec/dnxhddec.c |

[FFmpeg-devel] [PATCH 5/5] dnxhddec: replace some 0s by DNXHD_VARIABLE

2015-10-02 Thread Christophe Gisquet
A series of 0 in a table can be confusing, and the corresponding checks strange, so using a macro instead of that magic is more readable. --- libavcodec/dnxhddata.c | 10 +- libavcodec/dnxhddata.h | 3 +++ libavcodec/dnxhddec.c | 6 -- 3 files changed, 12 insertions(+), 7 deletions(