public inbox for gdb-testers@sourceware.org help / color / mirror / Atom feed
From: gdb-buildbot@sergiodj.net To: gdb-testers@sourceware.org Subject: [binutils-gdb] Create xml-builtin.h to declare xml_builtins Date: Tue, 22 Oct 2019 13:39:00 -0000 [thread overview] Message-ID: <fec4e896d6834bc6d23b9d666910dce43f2a2184@gdb-build> (raw) *** TEST RESULTS FOR COMMIT fec4e896d6834bc6d23b9d666910dce43f2a2184 *** commit fec4e896d6834bc6d23b9d666910dce43f2a2184 Author: Christian Biesinger <cbiesinger@google.com> AuthorDate: Wed Oct 9 20:08:13 2019 -0500 Commit: Christian Biesinger <cbiesinger@google.com> CommitDate: Wed Oct 16 18:19:14 2019 +0200 Create xml-builtin.h to declare xml_builtins xml-builtin.c only has character arrays and no dependencies, so this creates a simple header file for that purpose so that gdbserver can include that instead of re-declaring xml_builtin. Despite the name, feature_to_c.sh is already specific to xml_builtins (it hardcodes the variable name), so making it always output the include for xml-builtin.h seems fine. gdb/ChangeLog: 2019-10-16 Christian Biesinger <cbiesinger@google.com> * Makefile.in: Add xml-builtin.h. * features/feature_to_c.sh: Add an include for xml-builtin.h to ensure that the compiler checks that the types match. * xml-builtin.h: New file. * xml-support.c (fetch_xml_builtin): Add missing const. * xml-support.h: Remove declaration of xml_builtins. gdb/gdbserver/ChangeLog: 2019-10-16 Christian Biesinger <cbiesinger@google.com> * server.c: Include xml-builtin.h. (get_xml_features): Don't declare xml_builtins here. Change-Id: I806ef0851c43ead90b545a11794e41f5e5178436 diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 8748257e01..e79f449231 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,12 @@ +2019-10-16 Christian Biesinger <cbiesinger@google.com> + + * Makefile.in: Add xml-builtin.h. + * features/feature_to_c.sh: Add an include for xml-builtin.h + to ensure that the compiler checks that the types match. + * xml-builtin.h: New file. + * xml-support.c (fetch_xml_builtin): Add missing const. + * xml-support.h: Remove declaration of xml_builtins. + 2019-10-16 Tom de Vries <tdevries@suse.de> PR tdep/25096 diff --git a/gdb/Makefile.in b/gdb/Makefile.in index af4b14d6e6..8fec099a1b 100644 --- a/gdb/Makefile.in +++ b/gdb/Makefile.in @@ -1425,6 +1425,7 @@ HFILES_NO_SRCDIR = \ x86-linux-nat.h \ x86-nat.h \ xcoffread.h \ + xml-builtin.h \ xml-support.h \ xml-syscall.h \ xml-tdesc.h \ diff --git a/gdb/features/feature_to_c.sh b/gdb/features/feature_to_c.sh index 2e7e0c72a8..ae83774fdf 100755 --- a/gdb/features/feature_to_c.sh +++ b/gdb/features/feature_to_c.sh @@ -32,6 +32,8 @@ if test -e "$output"; then exit 1 fi +echo '#include "xml-builtin.h"' >> $output + for input; do arrayname=xml_feature_`echo $input | sed 's,.*/,,; s/[-.]/_/g'` diff --git a/gdb/gdbserver/ChangeLog b/gdb/gdbserver/ChangeLog index 801a66c563..b2213cffcf 100644 --- a/gdb/gdbserver/ChangeLog +++ b/gdb/gdbserver/ChangeLog @@ -1,3 +1,8 @@ +2019-10-16 Christian Biesinger <cbiesinger@google.com> + + * server.c: Include xml-builtin.h. + (get_xml_features): Don't declare xml_builtins here. + 2019-10-15 Andrew Burgess <andrew.burgess@embecosm.com> * Makefile.in: Remove references to vec-ipa.o. diff --git a/gdb/gdbserver/server.c b/gdb/gdbserver/server.c index 0bfff04fd7..25a2be86fb 100644 --- a/gdb/gdbserver/server.c +++ b/gdb/gdbserver/server.c @@ -41,6 +41,9 @@ #include "gdbsupport/environ.h" #include "filenames.h" #include "gdbsupport/pathstuff.h" +#ifdef USE_XML +#include "xml-builtin.h" +#endif #include "gdbsupport/selftest.h" #include "gdbsupport/scope-exit.h" @@ -920,7 +923,6 @@ get_features_xml (const char *annex) #ifdef USE_XML { - extern const char *const xml_builtin[][2]; int i; /* Look for the annex. */ diff --git a/gdb/xml-builtin.h b/gdb/xml-builtin.h new file mode 100644 index 0000000000..972417df32 --- /dev/null +++ b/gdb/xml-builtin.h @@ -0,0 +1,28 @@ +/* Header file for builtin XML files. + + Copyright (C) 2019 Free Software Foundation, Inc. + + This file is part of GDB. + + This program is free software; you can redistribute it and/or modify + it under the terms of the GNU General Public License as published by + the Free Software Foundation; either version 3 of the License, or + (at your option) any later version. + + This program is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + GNU General Public License for more details. + + You should have received a copy of the GNU General Public License + along with this program. If not, see <http://www.gnu.org/licenses/>. */ + +#ifndef XML_BUILTIN_H +#define XML_BUILTIN_H + +/* The text of compiled-in XML documents, from xml-builtin.c + (generated). */ + +extern const char *const xml_builtin[][2]; + +#endif /* XML_BUILTIN_H */ diff --git a/gdb/xml-support.c b/gdb/xml-support.c index eaf99efa6b..915be76066 100644 --- a/gdb/xml-support.c +++ b/gdb/xml-support.c @@ -19,6 +19,7 @@ #include "defs.h" #include "gdbcmd.h" +#include "xml-builtin.h" #include "xml-support.h" #include "gdbsupport/filestuff.h" #include "safe-ctype.h" @@ -919,7 +920,7 @@ xml_process_xincludes (std::string &result, const char * fetch_xml_builtin (const char *filename) { - const char *(*p)[2]; + const char *const (*p)[2]; for (p = xml_builtin; (*p)[0]; p++) if (strcmp ((*p)[0], filename) == 0) diff --git a/gdb/xml-support.h b/gdb/xml-support.h index 5012767568..59b4467d08 100644 --- a/gdb/xml-support.h +++ b/gdb/xml-support.h @@ -44,11 +44,6 @@ LONGEST xml_builtin_xfer_partial (const char *filename, gdb_byte *readbuf, const gdb_byte *writebuf, ULONGEST offset, LONGEST len); -/* The text of compiled-in XML documents, from xml-builtin.c - (generated). */ - -extern const char *xml_builtin[][2]; - /* Support for XInclude. */ /* Callback to fetch a new XML file, based on the provided HREF. */
next reply other threads:[~2019-10-22 13:39 UTC|newest] Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-10-22 13:39 gdb-buildbot [this message] 2019-10-22 13:39 ` Failures on Ubuntu-Aarch64-m64, branch master gdb-buildbot 2019-10-22 15:24 ` Failures on Ubuntu-Aarch64-native-gdbserver-m64, " gdb-buildbot 2019-10-31 10:43 ` Failures on Fedora-i686, " gdb-buildbot 2019-10-31 11:32 ` Failures on Fedora-x86_64-m32, " gdb-buildbot 2019-10-31 11:34 ` Failures on Fedora-x86_64-cc-with-index, " gdb-buildbot 2019-10-31 14:01 ` Failures on Fedora-x86_64-native-gdbserver-m32, " gdb-buildbot 2019-10-31 15:33 ` Failures on Fedora-x86_64-native-gdbserver-m64, " gdb-buildbot
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=fec4e896d6834bc6d23b9d666910dce43f2a2184@gdb-build \ --to=gdb-buildbot@sergiodj.net \ --cc=gdb-testers@sourceware.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).