[FFmpeg-cvslog] r22091 - in trunk/libavcodec: ivi_common.c ivi_common.h

Michael Niedermayer michaelni
Sun Feb 28 18:58:43 CET 2010


On Sat, Feb 27, 2010 at 01:12:56PM +0100, kostya wrote:
> Author: kostya
> Date: Sat Feb 27 13:12:56 2010
> New Revision: 22091
> 
> Log:
> Make it clear that ff_ivi_init_static_vlc() does not need arguments
> 
> Modified:
>    trunk/libavcodec/ivi_common.c
>    trunk/libavcodec/ivi_common.h
> 
> Modified: trunk/libavcodec/ivi_common.c
> ==============================================================================
> --- trunk/libavcodec/ivi_common.c	Sat Feb 27 11:16:45 2010	(r22090)
> +++ trunk/libavcodec/ivi_common.c	Sat Feb 27 13:12:56 2010	(r22091)
> @@ -89,7 +89,7 @@ int ff_ivi_create_huff_from_desc(const I
>                      (flag ? INIT_VLC_USE_NEW_STATIC : 0) | INIT_VLC_LE);
>  }
>  
> -void ff_ivi_init_static_vlc()
> +void ff_ivi_init_static_vlc(void)

can we add this as -Werror= ?
its a reoccuring mistake that can lead to actual bugs due to lack of
argument type checking

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I am the wisest man alive, for I know one thing, and that is that I know
nothing. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20100228/a01b957d/attachment.pgp>



More information about the ffmpeg-cvslog mailing list