xserver: Branch 'master'

Keith Packard keithp at kemper.freedesktop.org
Thu Sep 11 18:18:38 PDT 2014


 os/log.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 7a0c79c8c478bf22ee71c1ea35886a6f555ef2bb
Author: Thomas Petazzoni <thomas.petazzoni at free-electrons.com>
Date:   Tue Sep 2 22:03:14 2014 +0200

    os/log: adjust gcc version conditions for #pragma
    
    In commit e67f2d7e0f9189beb2907fa06cff5ecc7f35f922 ("gcc 4.2.1 doesn't
    support #pragma GCC diagnostic ignored"), some compile time
    conditionals were added around the #pragma usage. Those conditionals
    ensure that the #pragma are not used on gcc <= 4.2.
    
    However, the usage of #pragma diagnostic inside functions was only
    added in gcc 4.6, and a build failure is therefore experienced with
    gcc 4.5:
    
    log.c: In function 'LogInit':
    log.c:199:9: error: #pragma GCC diagnostic not allowed inside functions
    log.c:201:9: warning: format not a string literal, argument types not checked
    log.c:212:9: error: #pragma GCC diagnostic not allowed inside functions
    log.c:214:17: warning: format not a string literal, argument types not checked
    
    $ ./host/usr/bin/powerpc-linux-gnu-gcc -v
    [...]
    gcc version 4.5.2 (Sourcery G++ Lite 2011.03-38)
    
    This patch therefore adjusts the compile time conditionals to make
    sure the #pragma is not used on gcc <= 4.5, and only used on gcc >=
    4.6.
    
    Signed-off-by: Thomas Petazzoni <thomas.petazzoni at free-electrons.com>
    Acked-by:  Keith Packard <keithp at keithp.com>
    Signed-off-by: Keith Packard <keithp at keithp.com>

diff --git a/os/log.c b/os/log.c
index 2a721b9..629021e 100644
--- a/os/log.c
+++ b/os/log.c
@@ -195,7 +195,7 @@ LogInit(const char *fname, const char *backup)
     char *logFileName = NULL;
 
     if (fname && *fname) {
-#if __GNUC__ > 4 || __GNUC__ == 4 && __GNUC_MINOR__ > 2
+#if __GNUC__ > 4 || __GNUC__ == 4 && __GNUC_MINOR__ > 5
 #pragma GCC diagnostic ignored "-Wformat-nonliteral"
 #endif
         if (asprintf(&logFileName, fname, display) == -1)
@@ -208,7 +208,7 @@ LogInit(const char *fname, const char *backup)
                 char *suffix;
                 char *oldLog;
 
-#if __GNUC__ > 4 || __GNUC__ == 4 && __GNUC_MINOR__ > 2
+#if __GNUC__ > 4 || __GNUC__ == 4 && __GNUC_MINOR__ > 5
 #pragma GCC diagnostic ignored "-Wformat-nonliteral"
 #endif
                 if ((asprintf(&suffix, backup, display) == -1) ||


More information about the xorg-commit mailing list