[FFmpeg-devel] [PATCH] fix warnings in tree.c test

[Benoit Fouet]

Hi,

On 03/31/2009 02:35 PM, Michael Niedermayer wrote:
> On Tue, Mar 31, 2009 at 09:47:17AM +0200, Benoit Fouet wrote:
>   
>> Hi,
>>
>> attached patch to $subj:
>> libavutil/tree.c: In function ?print?:
>> libavutil/tree.c:170: warning: format ?%4d? expects type ?int?, but
>> argument 6 has type ?void *?
>> libavutil/tree.c: In function ?cmp?:
>> libavutil/tree.c:178: warning: pointer of type ?void *? used in subtraction
>> libavutil/tree.c: In function ?main?:
>> libavutil/tree.c:198: warning: passing argument 3 of ?av_tree_insert?
>> from incompatible pointer type
>> libavutil/tree.c:204: warning: passing argument 3 of ?av_tree_insert?
>> from incompatible pointer type
>> libavutil/tree.c:205: warning: passing argument 3 of ?av_tree_find? from
>> incompatible pointer type
>> libavutil/tree.c:205: warning: assignment makes integer from pointer
>> without a cast
>>     
>
> patch ok if tested
>
> [...]
>   
>   

my sequence to test it:
$ svn di libavutil/tree.c > tree.c.diff
$ svn revert libavutil/tree.c
$ make libavutil/tree-test
$ ./libavutil/tree-test > before 2>&1
$ patch -p0 -i tree.c.diff
$ make libavutil/tree-test
$ ./libavutil/tree-test > after 2>&1
$ cmp before after && echo ok
ok

is that enough to test it ?

Ben