[Ffmpeg-devel] [PATCH] Getting rid of inlining failure warnings

[Panagiotis Issaris]

Hi,

On Thu, 2006-11-09 at 14:59 +0100, Panagiotis Issaris wrote:
> [...]
> I'll send a new patch later in which I will send smaller patches,
> removing the inline specifier only if no new warnings are introduced by
> doing so.

The attached patch gets rid of inlining failure warnings.

Total number of warnings before the patch: 1141
Number of inlining warning before the patch: 353

Total number of warnings after the patch: 579
Number of inlining warning after the patch: 72


 asv1.c                |    6 +++---
 cavs.c                |    2 +-
 ffv1.c                |    4 ++--
 golomb.h              |    2 +-
 h263.c                |    8 ++++----
 h264.c                |    6 +++---
 jpeg_ls.c             |    4 ++--
 motion_est.c          |    6 +++---
 motion_est_template.c |    4 ++--
 mpeg12.c              |    6 +++---
 mpegvideo.c           |   14 +++++++-------
 mpegvideo.h           |    4 ++--
 msmpeg4.c             |    8 ++++----
 snow.c                |    6 +++---
 svq3.c                |    2 +-
 vc1.c                 |    2 +-
 16 files changed, 42 insertions(+), 42 deletions(-)

Regression tests succeed. As inlining already failed, removing the
specifier should have no performance impact (at least on all compilers
where the same inlining warnings would have appeared...)

With friendly regards,
Takis
-- 
vCard: http://www.issaris.org/pi.vcf
Public key: http://www.issaris.org/pi.key
-------------- next part --------------
A non-text attachment was scrubbed...
Name: pi-20061109T153801-ffmpeg-inline.diff
Type: text/x-patch
Size: 17305 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20061109/6fefebb1/attachment.bin>