[FFmpeg-devel] [PATCH] Electronic Arts MAD Decoder
Aurelien Jacobs
aurel
Thu Jun 11 23:27:02 CEST 2009
On Wed, Jun 10, 2009 at 01:37:13PM +0100, M?ns Rullg?rd wrote:
> Diego Biurrun <diego at biurrun.de> writes:
>
> > On Wed, Jun 10, 2009 at 08:52:58PM +1000, Peter Ross wrote:
> >>
> >> Patch updated.
> >>
> >> --- libavcodec/eamad.c (revision 0)
> >> +++ libavcodec/eamad.c (revision 0)
> >> @@ -0,0 +1,323 @@
> >> +
> >> +static av_cold int mad_decode_init(AVCodecContext *avctx)
> >
> > All those mad_ prefixes for static functions are pointless.
>
> I disagree. When using a debugger, it helps if functions have unique
> names. Setting a breakpoint on, or merely seeing in a call stack, a
> name like "decode" is ambiguous at best. Unique names also help with
> editors capable of jumping to a named symbol (emacs, vi, and many
> others).
I agree with Mans. I find those prefix, and having unique names useful.
Aurel
More information about the ffmpeg-devel
mailing list