diff options
author | Jim Meyering <meyering@fb.com> | 2016-03-14 08:59:56 -0700 |
---|---|---|
committer | Jim Meyering <meyering@fb.com> | 2016-03-15 07:36:12 -0700 |
commit | c18b3699e1b8c4ad2739761f2b9c306ff2303322 (patch) | |
tree | 2715bf265d5468b2965af4646092aebf26a23136 /gl/lib | |
parent | e735d417fb2e5c7427b3622f2a78e65e450b49a8 (diff) | |
download | coreutils-c18b3699e1b8c4ad2739761f2b9c306ff2303322.tar.xz |
maint: remove gl/lib/reg*.c.diff; no longer needed
* gl/lib/regcomp.c.diff: Remove file, now that gnulib's
regcomp.c compiles regex.c with -Wno-unused-parameter.
* gl/lib/regex_internal.h.diff: Likewise.
* gl/lib/regex_internal.c.diff: This file induced a change to ensure
that the "Idx" type was unsigned and to remove a few "VAR < 0"
comparisons. These days, it is probably fine to stay in sync with
gnulib/glibc's copies
of these files, so remove these patches, too.
* gl/lib/regexec.c.diff: Likewise.
Prompted by a report by Assaf Gordon and a suggestion from Paul Eggert.
Diffstat (limited to 'gl/lib')
-rw-r--r-- | gl/lib/regcomp.c.diff | 68 | ||||
-rw-r--r-- | gl/lib/regex_internal.c.diff | 25 | ||||
-rw-r--r-- | gl/lib/regex_internal.h.diff | 14 | ||||
-rw-r--r-- | gl/lib/regexec.c.diff | 45 |
4 files changed, 0 insertions, 152 deletions
diff --git a/gl/lib/regcomp.c.diff b/gl/lib/regcomp.c.diff deleted file mode 100644 index 2f5107b10..000000000 --- a/gl/lib/regcomp.c.diff +++ /dev/null @@ -1,68 +0,0 @@ -diff --git i/lib/regcomp.c w/lib/regcomp.c -index 6de9b72..bd069dd 100644 ---- i/lib/regcomp.c -+++ w/lib/regcomp.c -@@ -523,8 +523,8 @@ weak_alias (__regcomp, regcomp) - from either regcomp or regexec. We don't use PREG here. */ - - size_t --regerror (int errcode, const regex_t *__restrict preg, char *__restrict errbuf, -- size_t errbuf_size) -+regerror (int errcode, const regex_t *__restrict preg _UNUSED_PARAMETER_, -+ char *__restrict errbuf, size_t errbuf_size) - { - const char *msg; - size_t msg_size; -@@ -1407,7 +1407,7 @@ calc_first (void *extra, bin_tree_t *node) - - /* Pass 2: compute NEXT on the tree. Preorder visit. */ - static reg_errcode_t --calc_next (void *extra, bin_tree_t *node) -+calc_next (void *extra _UNUSED_PARAMETER_, bin_tree_t *node) - { - switch (node->token.type) - { -@@ -2821,8 +2821,10 @@ build_range_exp (const reg_syntax_t syntax, - static reg_errcode_t - internal_function - # ifdef RE_ENABLE_I18N --build_collating_symbol (bitset_t sbcset, re_charset_t *mbcset, -- Idx *coll_sym_alloc, const unsigned char *name) -+build_collating_symbol (bitset_t sbcset, -+ re_charset_t *mbcset _UNUSED_PARAMETER_, -+ Idx *coll_sym_alloc _UNUSED_PARAMETER_, -+ const unsigned char *name) - # else /* not RE_ENABLE_I18N */ - build_collating_symbol (bitset_t sbcset, const unsigned char *name) - # endif /* not RE_ENABLE_I18N */ -@@ -3392,7 +3394,8 @@ parse_bracket_exp (re_string_t *regexp, re_dfa_t *dfa, re_token_t *token, - - static reg_errcode_t - parse_bracket_element (bracket_elem_t *elem, re_string_t *regexp, -- re_token_t *token, int token_len, re_dfa_t *dfa, -+ re_token_t *token, int token_len, -+ re_dfa_t *dfa _UNUSED_PARAMETER_, - reg_syntax_t syntax, bool accept_hyphen) - { - #ifdef RE_ENABLE_I18N -@@ -3479,8 +3482,9 @@ parse_bracket_symbol (bracket_elem_t *elem, re_string_t *regexp, - - static reg_errcode_t - #ifdef RE_ENABLE_I18N --build_equiv_class (bitset_t sbcset, re_charset_t *mbcset, -- Idx *equiv_class_alloc, const unsigned char *name) -+build_equiv_class (bitset_t sbcset, re_charset_t *mbcset _UNUSED_PARAMETER_, -+ Idx *equiv_class_alloc _UNUSED_PARAMETER_, -+ const unsigned char *name) - #else /* not RE_ENABLE_I18N */ - build_equiv_class (bitset_t sbcset, const unsigned char *name) - #endif /* not RE_ENABLE_I18N */ -@@ -3884,7 +3888,7 @@ free_token (re_token_t *node) - and its children. */ - - static reg_errcode_t --free_tree (void *extra, bin_tree_t *node) -+free_tree (void *extra _UNUSED_PARAMETER_, bin_tree_t *node) - { - free_token (&node->token); - return REG_NOERROR; diff --git a/gl/lib/regex_internal.c.diff b/gl/lib/regex_internal.c.diff deleted file mode 100644 index 2a72e0de7..000000000 --- a/gl/lib/regex_internal.c.diff +++ /dev/null @@ -1,25 +0,0 @@ -diff --git a/lib/regex_internal.c b/lib/regex_internal.c -index 93d7ee9..9b86a07 100644 ---- a/lib/regex_internal.c -+++ b/lib/regex_internal.c -@@ -17,6 +17,8 @@ - License along with the GNU C Library; if not, see - <http://www.gnu.org/licenses/>. */ - -+#include "verify.h" -+#include "intprops.h" - static void re_string_construct_common (const char *str, Idx len, - re_string_t *pstr, - RE_TRANSLATE_TYPE trans, bool icase, -@@ -1389,7 +1391,10 @@ static void - internal_function - re_node_set_remove_at (re_node_set *set, Idx idx) - { -- if (idx < 0 || idx >= set->nelem) -+ verify (! TYPE_SIGNED (Idx)); -+ /* if (idx < 0) -+ return; */ -+ if (idx >= set->nelem) - return; - --set->nelem; - for (; idx < set->nelem; idx++) diff --git a/gl/lib/regex_internal.h.diff b/gl/lib/regex_internal.h.diff deleted file mode 100644 index 66ea41714..000000000 --- a/gl/lib/regex_internal.h.diff +++ /dev/null @@ -1,14 +0,0 @@ -diff --git a/lib/regex_internal.h b/lib/regex_internal.h -index 0307a34..45c2d44 100644 ---- a/lib/regex_internal.h -+++ b/lib/regex_internal.h -@@ -868,7 +868,8 @@ re_string_wchar_at (const re_string_t *pstr, Idx idx) - - static int - internal_function __attribute__ ((pure, unused)) --re_string_elem_size_at (const re_string_t *pstr, Idx idx) -+re_string_elem_size_at (const re_string_t *pstr _UNUSED_PARAMETER_, -+ Idx idx _UNUSED_PARAMETER_) - { - # ifdef _LIBC - const unsigned char *p, *extra; diff --git a/gl/lib/regexec.c.diff b/gl/lib/regexec.c.diff deleted file mode 100644 index 1aa3c2477..000000000 --- a/gl/lib/regexec.c.diff +++ /dev/null @@ -1,45 +0,0 @@ -diff --git a/lib/regexec.c b/lib/regexec.c -index d3920c4..a8152b5 100644 ---- a/lib/regexec.c -+++ b/lib/regexec.c -@@ -17,6 +17,8 @@ - License along with the GNU C Library; if not, see - <http://www.gnu.org/licenses/>. */ - -+#include "verify.h" -+#include "intprops.h" - static reg_errcode_t match_ctx_init (re_match_context_t *cache, int eflags, - Idx n) internal_function; - static void match_ctx_clean (re_match_context_t *mctx) internal_function; -@@ -373,8 +375,11 @@ re_search_2_stub (struct re_pattern_buffer *bufp, - Idx len = length1 + length2; - char *s = NULL; - -- if (BE (length1 < 0 || length2 < 0 || stop < 0 || len < length1, 0)) -- return -2; -+ verify (! TYPE_SIGNED (Idx)); -+ if (BE (len < length1, 0)) -+ return -2; -+ /* if (BE (length1 < 0 || length2 < 0 || stop < 0, 0)) -+ return -2; */ - - /* Concatenate the strings. */ - if (length2 > 0) -@@ -423,11 +428,14 @@ re_search_stub (struct re_pattern_buffer *bufp, - Idx last_start = start + range; - - /* Check for out-of-range. */ -- if (BE (start < 0 || start > length, 0)) -- return -1; -+ verify (! TYPE_SIGNED (Idx)); -+ /* if (BE (start < 0, 0)) -+ return -1; */ -+ if (BE (start > length, 0)) -+ return -1; - if (BE (length < last_start || (0 <= range && last_start < start), 0)) - last_start = length; -- else if (BE (last_start < 0 || (range < 0 && start <= last_start), 0)) -+ else if (BE (/* last_start < 0 || */ (range < 0 && start <= last_start), 0)) - last_start = 0; - - lock_lock (dfa->lock); |