summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* tests: use "compare exp out", not "compare out exp"Jim Meyering2011-11-2920-38/+38
| | | | | | | | | | | | | | 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/'
* build: update gnulib submodule to latestJim Meyering2011-11-291-0/+0
|
* build: accommodate -Werror=suggest-attribute=pureJim Meyering2011-11-283-6/+8
| | | | | | | | | | | 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.
* build: fix "make check" error on OSF/1Bruno Haible2011-11-211-1/+1
| | | | | * tests/Makefile.am (TESTS_ENVIRONMENT): Test the value of the variable BASH_VERSION, not the literal ASH_VERSION.
* portability: work consistently on *BSD systemsJim Meyering2011-11-211-1/+11
| | | | | | | | * 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.
* maint: consistently use NULL, not 0, when comparing pointersJim Meyering2011-11-201-3/+3
| | | | * src/dfa.c (dfaanalyze): Compare trans[s] with NULL, not 0.
* maint: remove an avoidable #ifdef/#endif pairJim Meyering2011-11-201-4/+2
| | | | * src/dfa.c (dfaanalyze): Remove avoidable #ifdef around "{".
* tests: fix typo in last changeJim Meyering2011-11-201-1/+1
| | | | | | | | * 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: avoid unwarranted test failure on *BSD-based systemsJim Meyering2011-11-201-2/+3
| | | | | * tests/word-delim-multibyte (e_acute): Use a more portable representation of e-acute. Reported by Bruno Haible.
* maint: accommodate -Wdeclaration-after-statement, but only in dfa.c,Jim Meyering2011-11-191-2/+3
| | | | | | | | 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.
* maint: post-release administriviaJim Meyering2011-11-163-2/+5
| | | | | | * NEWS: Add header line for next release. * .prev-version: Record previous version. * cfg.mk (old_NEWS_hash): Auto-update.
* version 2.10v2.10Jim Meyering2011-11-161-1/+1
| | | | * NEWS: Record release date.
* build: update gnulib submodule to latestJim Meyering2011-11-161-0/+0
|
* maint: update bootstrap and init.sh from gnulibJim Meyering2011-11-132-16/+5
| | | | | * tests/init.sh: Update from gnulib. * bootstrap: Likewise.
* build: update gnulib for exclude-test fixesJim Meyering2011-11-121-0/+0
|
* tests: make our "export" replacement efficient with modern shellsJim Meyering2011-11-121-11/+15
| | | | | | | * 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: make the replacement export function more robustJim Meyering2011-11-121-3/+3
| | | | | * tests/Makefile.am (sed_quote_value): Also quote single quotes. Remove sed's -e options. Not needed.
* tests: fix test suite execution failure on OSF/1 5.1Bruno Haible2011-11-121-1/+14
| | | | | | * 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: don't assume that /bin/bash existsBruno Haible2011-11-121-1/+1
| | | | * tests/fedora: Run using /bin/sh, not /bin/bash.
* tests: avoid unwarranted failures due to SATAN's timeoutBruno Haible2011-11-121-0/+2
| | | | | * tests/init.cfg (require_timeout_): Also ensure that timeout exits with its child's exit status.
* build: fix compilation error on MSVC 9 to due Pexecute() declarationBruno Haible2011-11-121-9/+4
| | | | | | * src/pcresearch.c (WITHOUT_PCRE_NORETURN): Remove macro. (Pexecute): Replace abort() call with code that does not trigger GCC warnings.
* tests: fix high-bit-range test failure on OSF/1 5.1Bruno Haible2011-11-121-2/+2
| | | | | * tests/high-bit-range: Use octal escape instead of hexadecimal escape sequence.
* build: update gnulib for solaris test fixJim Meyering2011-11-111-0/+0
|
* build: update gnulib submodule to latestJim Meyering2011-11-101-0/+0
|
* maint: adjust the URL that will appear in the generated announcementJim Meyering2011-11-101-0/+4
| | | | | | * 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.
* build: stop distributing gzip'd releases; xz is enoughJim Meyering2011-11-062-1/+7
| | | | | * configure.ac (AM_INIT_AUTOMAKE): Add no-dist-gzip. * NEWS (Build-related): Mention that we're dropping .tar.gz.
* build: update gnulib submodule to latestJim Meyering2011-11-061-0/+0
|
* distcheck: ensure dist-hook fails if syntax-check failsStefano Lattarini2011-10-141-1/+1
| | | | | | * 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.
* build: update gnulib submodule to latestJim Meyering2011-10-122-3/+3
| | | | | This should fix a few portability problems, including one on HP-UX and a test-float failure on PPC, reported by Andreas Metzler.
* gitignore: merge top-level and tests/ .gitignore filesStefano Lattarini2011-10-102-5/+5
| | | | | | * tests/.gitignore: Remove; what little remained of its contents has been moved ... * .gitignore: ... here.
* tests: tiny simplification in TESTS_ENVIRONMENT definitionStefano Lattarini2011-10-101-1/+1
| | | | | * tests/Makefile.am (TESTS_ENVIRONMENT): Remove redundant use of `export'.
* tests: support development version of automake tooStefano Lattarini2011-10-103-1/+5
| | | | | | | | | | | | | | | | 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.
* dfa: don't mishandle high-bit bytes in a regexp with signed-charEli Zaretskii2011-10-042-2/+12
| | | | | | | | | | | 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.
* maint: dfa: simplify multi-byte-related conditionalsJim Meyering2011-09-164-21/+20
| | | | | | | | | | | | | | * 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);
* maint: dfa: simplify several expressionsJim Meyering2011-09-161-10/+9
| | | | | | | | * 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 &&".
* maint: dfa: avoid in-function "#if MBS_SUPPORT" testsJim Meyering2011-09-161-10/+6
| | | | | | * src/dfa.c (setbit_case_fold_c): Remove "#if MBS_SUPPORT" in favor of simple "if (MBS_SUPPORT ...". (dfaexec, addtok): Likewise.
* maint: ensure that MB_CUR_MAX is defined even when !MBS_SUPPORTJim Meyering2011-09-161-0/+5
| | | | * src/mbsupport.h [!MBS_SUPPORT] (MB_CUR_MAX): Define to 1.
* build: fix compilation failure when MBS_SUPPORT is 0Jim Meyering2011-09-161-6/+6
| | | | | | | * 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.
* maint: dfa: simplify dfaoptimizeJim Meyering2011-09-161-22/+18
| | | | | | * src/dfa.c (dfaoptimize): Simplify. (dfacomp): Remove now-redundant "if (MBS_SUPPORT)" guard, since dfaoptimize does nothing if !MBS_SUPPORT.
* maint: dfa: remove some #if MBS_SUPPORT guardsJim Meyering2011-09-161-11/+6
| | | | | * src/dfa.c: Replace a few "#if MBS_SUPPORT" directives with "if (MBS_SUPPORT)". Remove some altogether.
* maint: dfa: convert #if-MBS_SUPPORT (dfastate)Jim Meyering2011-09-161-5/+3
| | | | * src/dfa.c (dfastate): Use regular "if", not #if MBS_SUPPORT.
* maint: dfa: convert #if-MBS_SUPPORT (dfastate)Jim Meyering2011-09-161-10/+5
| | | | * src/dfa.c (dfastate): Use regular "if", not #if MBS_SUPPORT.
* maint: dfa: convert #if-MBS_SUPPORT (state_index)Jim Meyering2011-09-161-4/+5
| | | | * src/dfa.c (state_index): Use regular "if", not #if MBS_SUPPORT.
* maint: dfa: convert #if-MBS_SUPPORT (dfaparse)Jim Meyering2011-09-161-3/+1
| | | | * src/dfa.c (dfaparse): Use regular "if", not #if MBS_SUPPORT.'
* maint: dfa: convert #if-MBS_SUPPORT (copytoks)Jim Meyering2011-09-161-14/+8
| | | | * src/dfa.c (copytoks): Use regular "if", not #if MBS_SUPPORT.'
* maint: dfa: convert #if-MBS_SUPPORT (lex)Jim Meyering2011-09-161-11/+4
| | | | * src/dfa.c (lex): Use regular "if", not #if MBS_SUPPORT.'
* maint: dfa: convert #if-MBS_SUPPORT (parse_bracket_exp)Jim Meyering2011-09-161-3/+1
| | | | * src/dfa.c (parse_bracket_exp): Use regular "if", not #if MBS_SUPPORT.
* maint: dfa: convert #if-MBS_SUPPORT (parse_bracket_exp)Jim Meyering2011-09-161-20/+19
| | | | * src/dfa.c (parse_bracket_exp): Use regular "if", not #if MBS_SUPPORT.
* maint: dfa: convert #if-MBS_SUPPORT (parse_bracket_exp)Jim Meyering2011-09-161-6/+2
| | | | * src/dfa.c (parse_bracket_exp): Use regular "if", not #if MBS_SUPPORT.
* maint: dfa: convert #if-MBS_SUPPORT (dfaexec)Jim Meyering2011-09-161-3/+1
| | | | * src/dfa.c (dfaexec): Use regular "if", not #if MBS_SUPPORT.