This is the mail archive of the gdb-cvs@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

gdb and binutils branch master updated. 355e210214ba52d477f25b33de00dbbcfb566c70


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "gdb and binutils".

The branch, master has been updated
       via  355e210214ba52d477f25b33de00dbbcfb566c70 (commit)
      from  1dfc6506b7482baf86dbb3c91de5bd2ae3f5acda (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=355e210214ba52d477f25b33de00dbbcfb566c70

commit 355e210214ba52d477f25b33de00dbbcfb566c70
Author: Joel Brobecker <brobecker@adacore.com>
Date:   Mon Oct 6 14:50:46 2014 -0400

    common-defs.h: include <stdarg.h> before <stdio.h>
    
    When trying to build gdbserver on ppc-lynx178, the compiler reports
    while trying to compile gdbserver/ax.c that vsprintf is not declared.
    Looking at my C99 reference manual (a draft), I see the following
    synopsis:
    
        #include <stdarg.h>
        #include <stdio.h>
        int vsprintf(char * restrict s, [etc]);
    
    Looking at stdio.h on LynxOS-178, if found where vsprintf gets
    declared:
    
        #if defined(__varargs_h) || defined(__stdarg_h) \
                || defined(_VARARGS_H) || defined(_STDARG_H)
        extern int vsprintf             _AP((char *, const char *, va_list));
        #endif
    
    Digging further, I noticed that common-defs.h, which is included
    via server.h, includes stdarg.h after including stdio, explaining
    why vsprintf does not get declared in this case.
    
    This patch fixes the problem by including stdarg.h before stdio.h.
    
    gdb/ChangeLog:
    
            * common/common-defs.h: Move <stdarg.h> #include ahead of
            <stdio.h> #include.
    
    Tested on x86_64-linux.

-----------------------------------------------------------------------

Summary of changes:
 gdb/ChangeLog            |    5 +++++
 gdb/common/common-defs.h |    2 +-
 2 files changed, 6 insertions(+), 1 deletions(-)


hooks/post-receive
-- 
gdb and binutils


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]