[FFmpeg-devel] [PATCH v3] Add support for Audible AAX (and AAX+) files

[Carl Eugen Hoyos]

Vesselin Bontchev <vesselin.bontchev <at> yandex.com> writes:

> +    if (!strncmp((char*)&type, "aax", 3)) {

Three lines above you have:
    if (strcmp(type, "qt  "))
        c->isom = 1;

So please make this:
..
 else if (!strcmp(type, "aax ")) {
   c->aax = 1;
 }
(type[4] is initialized to 0 and o
nly the 
first four bytes are overwritten.)

> +av_log(c->fc, AV_LOG_DEBUG, "[aax] aax file detected!\n");

The debug line should not be necessary because 
such a av_log() is already there.

Is hex_encode() the same as format specifier %x?

Carl Eugen