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

Ronald S. Bultje rsbultje
Tue Feb 8 22:02:41 CET 2011


Hi,

2011/2/8 M?ns Rullg?rd <mans at mansr.com>:
> Anton Khirnov <anton at khirnov.net> writes:
>
>> ---
>> ?libavformat/asf.c ? ?| ? ?4 ++++
>> ?libavformat/asfdec.c | ? ?5 -----
>> ?2 files changed, 4 insertions(+), 5 deletions(-)
>>
>> diff --git a/libavformat/asf.c b/libavformat/asf.c
>> index 1f6af18..10fa422 100644
>> --- a/libavformat/asf.c
>> +++ b/libavformat/asf.c
>> @@ -20,6 +20,10 @@
>>
>> ?#include "asf.h"
>>
>> +int ff_guidcmp(const void *g1, const void *g2)
>> +{
>> + ? ?return memcmp(g1, g2, sizeof(ff_asf_guid));
>> +}
>
> 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.

+1.

Ronald



More information about the ffmpeg-devel mailing list