[FFmpeg-devel] flac decoding
Fri Jan 23 23:38:32 CET 2009
Diego Biurrun wrote:
> On Thu, Jan 22, 2009 at 11:13:28PM -0500, Justin Ruggles wrote:
>> 1. rename flac.c to flacdec.c
> If it only contains decoding code, go right ahead.
>> 3. clean up debugging statements
>> flac.c is littered with debugging code. Some of it is commented-out,
>> some of it is not. Some of it is outdated. Should the debugging
>> printouts be left in there and the commented ones enabled? comment-out
>> the enabled ones? or take them out altogether. I have not found them
>> useful, but then again they sort of do the same as "flac -a" and it
>> might be useful at some point to compare the values in the two outputs.
> I say remove them. They will be easy to restore in case of need.
>> 4. remove unused code
>> There are some chunks of code which are disabled with ifdefs or
>> commented out. They seem to mostly be alternative implementations...
>> What is being used seems to work ok. I see no need to keep this old code
>> around in HEAD.
> I vote for removal.
I will do that soon unless there are objections.
>> 5. coding style clean up & some indentation cosmetics
>> The coding style in flac.c is very inconsistent.
> I think as long as you keep this properly separated from other types
> of changes, you can always directly commit such things without asking.
> Oh, and I suggest going for K&R style while you're at it...
Ok. I will do this as a single commit separate from any other cosmetic
>> Suggestions welcome!
> Take over maintainership.
I will add my name to the MAINTAINERS list for flacdec.c
More information about the ffmpeg-devel