[MPlayer-dev-eng] [PATCH] improved AIFF PCM Audio support - take 5
Reimar Döffinger
Reimar.Doeffinger at stud.uni-karlsruhe.de
Sun Dec 19 12:53:02 CET 2004
Hi,
> Thank you for the clarification. Attached is the revised patch which
> eliminates the backward-seek. Please review and accept. Thank you
Sorry, got a bit lost in all those pending patches and other problems.
Our current rule is that all functions and global variables must be commented with
doxygen-comatible comments (you should make the comments at the top of
the file in that style too I'd say). See
DOCS/tech/code-documentation.txt for a short explanation of how this
works.
[...]
> + printf("---------------------------------------\n");
> + printf("FORM id %c%c%c%c\n", f->fId[0], f->fId[1], f->fId[2], f->fId[3]);
> + printf("FORM size : %d\n",f->fSize);
[...]
better use mp_msg or at least print it to stderr I'd say...
> + if(sh->ah) free(sh->ah);
> + // aiff codec parameters:
> + AIFF_HEADER *ah;
> // codec-specific:
> void* context; // codec-specific stuff (usually HANDLE or struct pointer)
> unsigned char* codecdata; // extra header data passed from demuxer to codec
Please use the codecdata and codecdata_len instead of adding something
new.
Greetings,
Reimar Döffinger
More information about the MPlayer-dev-eng
mailing list