[06/17] Regex: Use re_malloc / re_free consistently.
Commit Message
arnold@skeeve.com wrote:
> Paul --- I think that you have permission to push the patches you approve
> to glibc. Please do so.
OK, I just now did that for patch 04/17, using a variant of Eric Blake's
original commit message for Gnulib (see attached). Carlos already installed
patch 01/17. As discussed, 02, 03, 05, 06, and 07 need thought before they can
go in, and the other patches I haven't reviewed yet. I'd like to finish
reviewing them all before installing patches that need thought.
From bff2701c1234446e4e873bee8894f6650a9dc3b3 Mon Sep 17 00:00:00 2001
From: Eric Blake <ebb9@byu.net>
Date: Fri, 22 Dec 2017 07:57:25 -0800
Subject: [PATCH] Avoid gcc warnings on cygwin
* posix/regex_internal.c (re_string_reconstruct) [!RE_ENABLE_I18N]:
* posix/regexec.c (check_arrival_add_next_nodes) [!RE_ENABLE_I18N]:
Avoid unused variable.
---
ChangeLog | 7 +++++++
posix/regex_internal.c | 2 +-
posix/regexec.c | 2 ++
3 files changed, 10 insertions(+), 1 deletion(-)
@@ -1,3 +1,10 @@
+2017-12-22 Eric Blake <ebb9@byu.net>
+
+ Avoid gcc warnings on cygwin
+ * posix/regex_internal.c (re_string_reconstruct) [!RE_ENABLE_I18N]:
+ * posix/regexec.c (check_arrival_add_next_nodes) [!RE_ENABLE_I18N]:
+ Avoid unused variable.
+
2017-12-22 Florian Weimer <fweimer@redhat.com>
* io/Makefile (routines): Add copy_file_range.
@@ -681,10 +681,10 @@ re_string_reconstruct (re_string_t *pstr, int idx, int eflags)
}
else
{
+#ifdef RE_ENABLE_I18N
/* No, skip all characters until IDX. */
int prev_valid_len = pstr->valid_len;
-#ifdef RE_ENABLE_I18N
if (BE (pstr->offsets_needed, 0))
{
pstr->len = pstr->raw_len - idx + offset;
@@ -2988,7 +2988,9 @@ check_arrival_add_next_nodes (re_match_context_t *mctx, int str_idx,
const re_dfa_t *const dfa = mctx->dfa;
int result;
int cur_idx;
+#ifdef RE_ENABLE_I18N
reg_errcode_t err = REG_NOERROR;
+#endif
re_node_set union_set;
re_node_set_init_empty (&union_set);
for (cur_idx = 0; cur_idx < cur_nodes->nelem; ++cur_idx)
--
2.14.3