From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 11072 invoked by alias); 8 May 2013 20:57:20 -0000 Mailing-List: contact archer-commits-help@sourceware.org; run by ezmlm Sender: Precedence: bulk List-Post: List-Help: List-Subscribe: Received: (qmail 11052 invoked by uid 306); 8 May 2013 20:57:20 -0000 Date: Wed, 08 May 2013 20:57:00 -0000 Message-ID: <20130508205719.11022.qmail@sourceware.org> From: tromey@sourceware.org To: archer-commits@sourceware.org Subject: [SCM] tromey/cleanup-checker: fix up xml-support.c X-Git-Refname: refs/heads/tromey/cleanup-checker X-Git-Reftype: branch X-Git-Oldrev: 6d3278189a492c717aa0cee73d40abff79cfd1c7 X-Git-Newrev: ecc236fbd1bf2e02ba923072ec1861f17ca0b369 X-SW-Source: 2013-q2/txt/msg00060.txt.bz2 List-Id: The branch, tromey/cleanup-checker has been updated via ecc236fbd1bf2e02ba923072ec1861f17ca0b369 (commit) via 5aa931fe2c861e072c98d315736fb0e9a917548d (commit) from 6d3278189a492c717aa0cee73d40abff79cfd1c7 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email. - Log ----------------------------------------------------------------- commit ecc236fbd1bf2e02ba923072ec1861f17ca0b369 Author: Tom Tromey Date: Wed May 8 14:57:11 2013 -0600 fix up xml-support.c commit 5aa931fe2c861e072c98d315736fb0e9a917548d Author: Tom Tromey Date: Wed May 8 14:52:26 2013 -0600 fix compile_rx_or_error ----------------------------------------------------------------------- Summary of changes: gdb/probe.c | 9 ++++++--- gdb/utils.c | 7 ++----- gdb/xml-support.c | 39 ++++++++++++--------------------------- gdb/xml-support.h | 7 ------- 4 files changed, 20 insertions(+), 42 deletions(-) First 500 lines of diff: diff --git a/gdb/probe.c b/gdb/probe.c index 05bdd1b..3086f4d 100644 --- a/gdb/probe.c +++ b/gdb/probe.c @@ -245,9 +245,12 @@ collect_probes (char *objname, char *provider, char *probe_name, cleanup = make_cleanup (VEC_cleanup (probe_p), &result); cleanup_temps = make_cleanup (null_cleanup, NULL); - compile_rx_or_error (&prov_pat, provider, _("Invalid provider regexp")); - compile_rx_or_error (&probe_pat, probe_name, _("Invalid probe regexp")); - compile_rx_or_error (&obj_pat, objname, _("Invalid object file regexp")); + if (provider != NULL) + compile_rx_or_error (&prov_pat, provider, _("Invalid provider regexp")); + if (probe_name != NULL) + compile_rx_or_error (&probe_pat, probe_name, _("Invalid probe regexp")); + if (objname != NULL) + compile_rx_or_error (&obj_pat, objname, _("Invalid object file regexp")); ALL_OBJFILES (objfile) { diff --git a/gdb/utils.c b/gdb/utils.c index c25dadf..fa54e10 100644 --- a/gdb/utils.c +++ b/gdb/utils.c @@ -1127,17 +1127,14 @@ get_regcomp_error (int code, regex_t *rx) } /* Compile a regexp and throw an exception on error. This returns a - cleanup to free the resulting pattern on success. If RX is NULL, - this does nothing and returns NULL. */ + cleanup to free the resulting pattern on success. RX must not be + NULL. */ struct cleanup * compile_rx_or_error (regex_t *pattern, const char *rx, const char *message) { int code; - if (!rx) - return NULL; - code = regcomp (pattern, rx, REG_NOSUB); if (code != 0) { diff --git a/gdb/xml-support.c b/gdb/xml-support.c index b777814..7fdbf7e 100644 --- a/gdb/xml-support.c +++ b/gdb/xml-support.c @@ -440,17 +440,18 @@ gdb_xml_cleanup (void *arg) xfree (parser); } -/* Initialize and return a parser. Register a cleanup to destroy the - parser. */ +/* Initialize a parser and store it to *PARSER_RESULT. Register a + cleanup to destroy the parser. */ -static struct gdb_xml_parser * +static struct cleanup * gdb_xml_create_parser_and_cleanup_1 (const char *name, const struct gdb_xml_element *elements, - void *user_data, struct cleanup **old_chain) + void *user_data, + struct gdb_xml_parser **parser_result) { struct gdb_xml_parser *parser; struct scope_level start_scope; - struct cleanup *dummy; + struct cleanup *result; /* Initialize the parser. */ parser = XZALLOC (struct gdb_xml_parser); @@ -476,25 +477,8 @@ gdb_xml_create_parser_and_cleanup_1 (const char *name, start_scope.elements = elements; VEC_safe_push (scope_level_s, parser->scopes, &start_scope); - if (old_chain == NULL) - old_chain = &dummy; - - *old_chain = make_cleanup (gdb_xml_cleanup, parser); - return parser; -} - -/* Initialize and return a parser. Register a cleanup to destroy the - parser. */ - -struct gdb_xml_parser * -gdb_xml_create_parser_and_cleanup (const char *name, - const struct gdb_xml_element *elements, - void *user_data) -{ - struct cleanup *old_chain; - - return gdb_xml_create_parser_and_cleanup_1 (name, elements, user_data, - &old_chain); + *parser_result = parser; + return make_cleanup (gdb_xml_cleanup, parser); } /* External entity handler. The only external entities we support @@ -623,8 +607,8 @@ gdb_xml_parse_quick (const char *name, const char *dtd_name, struct cleanup *back_to; int result; - parser = gdb_xml_create_parser_and_cleanup_1 (name, elements, - user_data, &back_to); + back_to = gdb_xml_create_parser_and_cleanup_1 (name, elements, + user_data, &parser); if (dtd_name != NULL) gdb_xml_use_dtd (parser, dtd_name); result = gdb_xml_parse (parser, document); @@ -897,7 +881,8 @@ xml_process_xincludes (const char *name, const char *text, obstack_init (&data->obstack); back_to = make_cleanup (xml_xinclude_cleanup, data); - parser = gdb_xml_create_parser_and_cleanup (name, xinclude_elements, data); + gdb_xml_create_parser_and_cleanup_1 (name, xinclude_elements, + data, &parser); parser->is_xinclude = 1; data->include_depth = depth; diff --git a/gdb/xml-support.h b/gdb/xml-support.h index a319678..a3a15ca 100644 --- a/gdb/xml-support.h +++ b/gdb/xml-support.h @@ -171,13 +171,6 @@ struct gdb_xml_element gdb_xml_element_end_handler *end_handler; }; -/* Initialize and return a parser. Register a cleanup to destroy the - parser. */ - -struct gdb_xml_parser *gdb_xml_create_parser_and_cleanup - (const char *name, const struct gdb_xml_element *elements, - void *user_data); - /* Associate DTD_NAME, which must be the name of a compiled-in DTD, with PARSER. */ hooks/post-receive -- Repository for Project Archer.