| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Likewise, when an empty file is expected, use "compare /dev/null out",
not "compare out /dev/null". I.e., specify the expected/desired contents
via the first file name. Prompted by a suggestion from Bruno Haible
in http://thread.gmane.org/gmane.comp.gnu.grep.bugs/4020/focus=29154
Run these commands:
git grep -l -E 'compare [^ ]+ exp' \
|xargs perl -pi -e 's/(compare) (\S+) (exp\S*)/$1 $3 $2/'
git grep -l -E 'compare [^ ]+ /dev/null' \
|xargs perl -pi -e 's/(compare) (\S+) (\/dev\/null)/$1 $3 $2/'
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
Now that we're using the latest manywarnings module from gnulib,
accommodate gcc's -Werror=suggest-attribute=pure option by marking
suggested functions with gnulib-defined _GL_ATTRIBUTE_PURE.
* src/kwset.c (hasevery): Mark function with pure attribute.
(bmexec): Likewise.
* src/dfa.c (nsubtoks, istrstr, find_pred, dfamusts): Likewise.
* configure.ac: Disable (for lib/) options that seem not to be worth
the trouble: -Wunsuffixed-float-constants and -Wformat-nonliteral.
|
|
|
|
|
| |
* tests/Makefile.am (TESTS_ENVIRONMENT): Test the value of the variable
BASH_VERSION, not the literal ASH_VERSION.
|
|
|
|
|
|
|
|
| |
* src/dfa.c (is_valid_unibyte_character): Define.
(IS_WORD_CONSTITUENT): Use it here, to make grep work consistently
even on *BSD systems, which use different tables for ctype macros
like isalpha. http://thread.gmane.org/gmane.comp.gnu.grep.bugs/4022
With help from Bruno Haible.
|
|
|
|
| |
* src/dfa.c (dfaanalyze): Compare trans[s] with NULL, not 0.
|
|
|
|
| |
* src/dfa.c (dfaanalyze): Remove avoidable #ifdef around "{".
|
|
|
|
|
|
|
|
| |
* tests/word-delim-multibyte: Use double quotes around $e_acute,
not single quotes. Spotted by Bruno Haible.
This and the preceding change do not resolve the XPASS failure
on OpenBSD 4.9 after all. See the explanation at
http://thread.gmane.org/gmane.comp.gnu.grep.bugs/4022
|
|
|
|
|
| |
* tests/word-delim-multibyte (e_acute): Use a more portable
representation of e-acute. Reported by Bruno Haible.
|
|
|
|
|
|
|
|
| |
and because doing so does not impact readability/maintainability.
This is solely to accommodate gawk users who are stuck with ancient gcc.
This is no excuse to change any other code in grep.
* src/dfa.c (dfaoptimize, parse_bracket_exp): Move declaration
to precede first statement in block.
|
|
|
|
|
|
| |
* NEWS: Add header line for next release.
* .prev-version: Record previous version.
* cfg.mk (old_NEWS_hash): Auto-update.
|
|
|
|
| |
* NEWS: Record release date.
|
| |
|
|
|
|
|
| |
* tests/init.sh: Update from gnulib.
* bootstrap: Likewise.
|
| |
|
|
|
|
|
|
|
| |
* tests/Makefile.am (TESTS_ENVIRONMENT): Use a trivial and efficient
implementation with a shell that supports "export var=val".
Use the sed-invoking replacement only when necessary.
Improved by Stefano Lattarini.
|
|
|
|
|
| |
* tests/Makefile.am (sed_quote_value): Also quote single quotes.
Remove sed's -e options. Not needed.
|
|
|
|
|
|
| |
* tests/Makefile.am (TESTS_ENVIRONMENT): Use a shell function to
ensure that we use only the portable form of the 'export' shell
built-in.
|
|
|
|
| |
* tests/fedora: Run using /bin/sh, not /bin/bash.
|
|
|
|
|
| |
* tests/init.cfg (require_timeout_): Also ensure that
timeout exits with its child's exit status.
|
|
|
|
|
|
| |
* src/pcresearch.c (WITHOUT_PCRE_NORETURN): Remove macro.
(Pexecute): Replace abort() call with code that does not trigger GCC
warnings.
|
|
|
|
|
| |
* tests/high-bit-range: Use octal escape instead of hexadecimal escape
sequence.
|
| |
|
| |
|
|
|
|
|
|
| |
* cfg.mk (url_dir_list): Use this http://ftp.gnu.org/gnu/$(PACKAGE)
for the first link listed in the generated announcement.
announce-gen now provides the faster mirror link automatically.
|
|
|
|
|
| |
* configure.ac (AM_INIT_AUTOMAKE): Add no-dist-gzip.
* NEWS (Build-related): Mention that we're dropping .tar.gz.
|
| |
|
|
|
|
|
|
| |
* Makefile.am (run-syntax-check): Fix logic, to ensure that
the recipe of this target returns a non-zero exit status if
"make syntax-check" fails.
|
|
|
|
|
| |
This should fix a few portability problems, including one on HP-UX
and a test-float failure on PPC, reported by Andreas Metzler.
|
|
|
|
|
|
| |
* tests/.gitignore: Remove; what little remained of its
contents has been moved ...
* .gitignore: ... here.
|
|
|
|
|
| |
* tests/Makefile.am (TESTS_ENVIRONMENT): Remove redundant use of
`export'.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change implements a more correct and idiomatic use of the
features of the Automake-provided 'parallel-tests' harness.
Moreover, this change is required in order for the testsuite to
continue to work with the new testsuite harness that is planned
to be introduced in Automake 1.12 (which, as of the writing date,
is still under development and in late alpha state).
* tests/Makefile.am (TESTS_ENVIRONMENT): The development version of
automake dos not support setting the interpreter delegated to run
the tests scripts in this variable; instead, use ...
(LOG_COMPILER): ... this variable.
* .gitignore: Ignore `.trs' files in directory `tests/'.
* build-aux/.gitignore: Ignore `test-driver' script.
|
|
|
|
|
|
|
|
|
|
|
| |
This appears to arise only on systems for which "char" is signed.
* src/dfa.c (FETCH_WC, FETCH): Produce an unsigned value, rather
than a sign-extended one. Fixes a bug on MS-Windows with compiling
patterns that include characters with the 8-th bit set.
(to_uchar): Define. From coreutils.
Reported by David Millis <tvtronix@yahoo.com>.
See http://thread.gmane.org/gmane.comp.gnu.grep.bugs/3893
* NEWS (Bug fixes): Mention it.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* src/dfa.c (setbit_case_fold_c, parse_bracket_exp, lex):
(addtok_mb, dfaparse): Change each "MBS_SUPPORT && MB_CUR_MAX > 1"
test to just "MB_CUR_MAX > 1".
* src/dfasearch.c (kwsincr_case, EGexecute): Likewise.
* src/kwsearch.c (Fcompile, Fexecute): Likewise.
* src/searchutils.c (kwsinit): Likewise.
* src/dfa.c (parse_bracket_exp): Convert
"if (!MBS_SUPPORT || MB_CUR_MAX == 1)" to
"if (MB_CUR_MAX == 1)" and do this:
- assert(!MBS_SUPPORT || MB_CUR_MAX == 1);
+ assert(MB_CUR_MAX == 1);
|
|
|
|
|
|
|
|
| |
* src/dfa.c (dfainit): Set d->mb_cur_max unconditionally, now
that MB_CUR_MAX is always usable. With that, simplify all
"MBS_SUPPORT && d->mb_cur_max > 1" to simply "d->mb_cur_max > 1".
(dfastate, dfaexec, dfainit, dfafree): Simplify, removing each
now-unnecessary "MBS_SUPPORT &&".
|
|
|
|
|
|
| |
* src/dfa.c (setbit_case_fold_c): Remove "#if MBS_SUPPORT" in favor
of simple "if (MBS_SUPPORT ...".
(dfaexec, addtok): Likewise.
|
|
|
|
| |
* src/mbsupport.h [!MBS_SUPPORT] (MB_CUR_MAX): Define to 1.
|
|
|
|
|
|
|
| |
* src/dfa.c (add_utf8_anychar): Always compile this function,
but when MBS_SUPPORT is 0, give it an empty body.
(prepare_wc_buf): Likewise.
[! MBS_SUPPORT] (setbit_wc): Define to always abort.
|
|
|
|
|
|
| |
* src/dfa.c (dfaoptimize): Simplify.
(dfacomp): Remove now-redundant "if (MBS_SUPPORT)" guard,
since dfaoptimize does nothing if !MBS_SUPPORT.
|
|
|
|
|
| |
* src/dfa.c: Replace a few "#if MBS_SUPPORT" directives with
"if (MBS_SUPPORT)". Remove some altogether.
|
|
|
|
| |
* src/dfa.c (dfastate): Use regular "if", not #if MBS_SUPPORT.
|
|
|
|
| |
* src/dfa.c (dfastate): Use regular "if", not #if MBS_SUPPORT.
|
|
|
|
| |
* src/dfa.c (state_index): Use regular "if", not #if MBS_SUPPORT.
|
|
|
|
| |
* src/dfa.c (dfaparse): Use regular "if", not #if MBS_SUPPORT.'
|
|
|
|
| |
* src/dfa.c (copytoks): Use regular "if", not #if MBS_SUPPORT.'
|
|
|
|
| |
* src/dfa.c (lex): Use regular "if", not #if MBS_SUPPORT.'
|
|
|
|
| |
* src/dfa.c (parse_bracket_exp): Use regular "if", not #if MBS_SUPPORT.
|
|
|
|
| |
* src/dfa.c (parse_bracket_exp): Use regular "if", not #if MBS_SUPPORT.
|
|
|
|
| |
* src/dfa.c (parse_bracket_exp): Use regular "if", not #if MBS_SUPPORT.
|
|
|
|
| |
* src/dfa.c (dfaexec): Use regular "if", not #if MBS_SUPPORT.
|