[FFmpeg-devel] [PATCH 01/10] lavf: move ff_guidcmp from asfdec.c to asf.c

[Måns Rullgård]

Diego Elio Petten? <flameeyes at gmail.com> writes:

> Il giorno mar, 08/02/2011 alle 21.00 +0000, M?ns Rullg?rd ha scritto:
>> 
>> Why is this a function at all, and why is it in asf.c?  If calling
>> memcmp directly is too much trouble, I'd at make it a macro instead. 
>
> Let's remember that while inlining has overhead, exposing new symbols
> has its share of overhead as well, especially as long as we don't hide
> internal symbols.

Inlining of that particular function has negative overhead.

-- 
M?ns Rullg?rd
mans at mansr.com