Diego Biurrun <di...@biurrun.de> writes:
> On Wed, Apr 18, 2012 at 06:07:49PM +0100, Måns Rullgård wrote:
>> Diego Biurrun <di...@biurrun.de> writes:
>> > --- a/libavcodec/dv_vlc_data.h
>> > +++ b/libavcodec/dv_vlc_data.h
>> > @@ -36,7 +36,7 @@
>> > */
>> > -static const uint16_t dv_vlc_bits[409] = {
>> > +static const uint16_t dv_vlc_bits[NB_DV_VLC] = {
>> > 0x0000, 0x0002, 0x0007, 0x0008, 0x0009, 0x0014, 0x0015, 0x0016,
>> > 0x0017, 0x0030, 0x0031, 0x0032, 0x0033, 0x0068, 0x0069, 0x006a,
>> > @@ -91,7 +91,7 @@ static const uint16_t dv_vlc_bits[409] = {
>> >
>> > -static const uint8_t dv_vlc_len[409] = {
>> > +static const uint8_t dv_vlc_len[NB_DV_VLC] = {
>> > 2, 3, 4, 4, 4, 5, 5, 5,
>> > 5, 6, 6, 6, 6, 7, 7, 7,
>> > @@ -146,7 +146,7 @@ static const uint8_t dv_vlc_len[409] = {
>> >
>> > -static const uint8_t dv_vlc_run[409] = {
>> > +static const uint8_t dv_vlc_run[NB_DV_VLC] = {
>> > 0, 0, 1, 0, 0, 2, 1, 0,
>> > 0, 3, 4, 0, 0, 5, 6, 2,
>> > @@ -201,7 +201,7 @@ static const uint8_t dv_vlc_run[409] = {
>> >
>> > -static const uint8_t dv_vlc_level[409] = {
>> > +static const uint8_t dv_vlc_level[NB_DV_VLC] = {
>> > 1, 2, 1, 3, 4, 1, 2, 5,
>> > 6, 1, 1, 7, 8, 1, 1, 2,
>>
>> Probably OK, unless those numbers are just coincidentally equal.
>
> The constant is used in dv_tablegen.h:dv_vlc_map_tableinit() to
> iterate over the table entries. That was enough to convince me.
In that case, patch OK.
--
Måns Rullgård
m...@mansr.com
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
