ralloc: Annotate printf functions with PRINTFLIKE(...)
authorMatt Turner <mattst88@gmail.com>
Tue, 23 Oct 2012 20:30:09 +0000 (13:30 -0700)
committerIan Romanick <ian.d.romanick@intel.com>
Fri, 2 Nov 2012 18:30:26 +0000 (11:30 -0700)
commit94cc306e41a2565b1dc7d410713ac78a9f1edaf3
tree7545fe11706078577799ecac108e06846099f4d1
parent4be397e164a495f9f2ae8b13497c44f5d584e85f
ralloc: Annotate printf functions with PRINTFLIKE(...)

This is a squash of the following two commits:

    ralloc: Annotate printf functions with PRINTFLIKE(...)

    Catches problems such as (in the gles3 branch)

    glcpp-parse.y: In function '_glcpp_parser_handle_version_declaration':
    glcpp-parse.y:1990:39: warning: format '%lli' expects argument of type
     'long long int', but argument 4 has type 'int' [-Wformat]

    As a side-effect, remove ralloc.c's likely/unlikely macros and just use
    the ones from main/compiler.h.

    NOTE: This is a candidate for the release branches.

Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
    (cherry picked from commit 41b14d125196e75cdd4599a4c1069b41e7000d48)

and

    src/glsl/tests/Makefile.am: Specify -I... in AM_CPPFLAGS

    When specifying per-target CFLAGS (e.g., ralloc_test_CFLAGS) AM_CFLAGS
    are not used. AM_CPPFLAGS should be used for includes anyway.

    Fixes a build problem since 41b14d125:

    CC       ralloc_test-ralloc.o
    In file included from ../../../src/glsl/ralloc.c:42:0:
    ../../../src/glsl/ralloc.h:57:27: fatal error: main/compiler.h: No such file or directory

Acked-by: Paul Berry <stereotype441@gmail.com>
    (cherry picked from commit 67f1e7bf5f5d1482cb8684dd5a405b7bab5b1f34)
src/glsl/ralloc.c
src/glsl/ralloc.h
src/glsl/tests/Makefile.am