[SCM] make branch, master, updated. 4.3-166-g56900846

[Paul D. Smith]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "make".

The branch, master has been updated
       via  5690084634b507ad7427589a9525eca05d15eb4f (commit)
       via  9fa63eb918e965d6704f33c1b124d14357af9911 (commit)
       via  668eda0527b9fb2aacb6fcb5a6124735afde3334 (commit)
       via  4e1be4a60c33bdfd99ef764af4bf1af359c8940a (commit)
      from  5b1e871d2dfc9195ad10b856b239211bd2972ee9 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 5690084634b507ad7427589a9525eca05d15eb4f
Author: Noah Goldstein <goldstein.w.n@gmail.com>
Date:   Sun Apr 24 17:43:56 2022 -0400

    Replace strcmp() with memcmp() where possible
    
    memcmp() is always faster than strcmp().  In places where we already
    know that both buffers have sufficient size, replace strcmp() with
    memcmp().
    
    * src/main.c (main): Replace strncmp() with memcmp()==0.
    * src/read.c (word1eq): Ditto.
    * src/commands.c (set_file_variables): Ditto.
    * src/function.c (func_filter_filterout): Ditto.
    (a_word_hash_cmp): Use STRING_N_COMPARE since we know the length.
    (func_sort): Replace strcmp() with memcmp().

commit 9fa63eb918e965d6704f33c1b124d14357af9911
Author: Noah Goldstein <goldstein.w.n@gmail.com>
Date:   Sun Apr 24 17:39:35 2022 -0400

    hash: Remove unnecessary isupper() check before tolower()
    
    The standard requires that tolower() returns its argument if there's
    no valid lowercase conversion: it's not necessary to check isupper()
    before invoking tolower().
    
    Comparing the performance of the old ISTRING_HASH to the new one
    on all the files present in GLIBC.
    
      Data Type            Old Time   New Time
      Alternating case    24985.754  13883.422
      random case         35211.777  13569.051
      all lower           18818.974  13706.645
      all upper           38859.454  13506.315
    
    * src/hash.h (ISTRING_HASH_1): Omit isupper() check.
    (ISTRING_HASH_2): Ditto.

commit 668eda0527b9fb2aacb6fcb5a6124735afde3334
Author: Dmitry Goncharov <dgoncharov@users.sf.net>
Date:   Sun Apr 24 17:10:46 2022 -0400

    [SV 62206] Fix %-substitution in second expansion of pattern rules
    
    During second expansion of pattern rules only the first pattern in
    each "group" was being substituted.  E.g. in this makefile:
    
      .SECONDEXPANSION:
      all: hello.x
      %.x: $$(wordlist 1, 99, %.1 %.%.2) ; $(info $@ from $^)
      hello.1 hello.\%.2 \%.1 \%.\%.2: ;
    
    the output would build "hello.1" and "%.%.2" because each function
    is considered a single "word" and only the first pattern is replaced.
    
    Fix the expansion so each whitespace-separated string is considered a
    word and the first pattern is replaced, giving "hello.1" and
    "hello.%.2".
    
    * src/rule.c (snap_implicit_rules): Keep enough space to replace %
    with $(*F) if necessary.
    * src/implicit.c (pattern_search): During second expansion break each
    get_next_word result into individual words and replace the first % in
    each with $* or $(*F) as needed.
    * tests/scripts/features/patternrules: Add tests for variations.

commit 4e1be4a60c33bdfd99ef764af4bf1af359c8940a
Author: Dmitry Goncharov <dgoncharov@users.sf.net>
Date:   Sun Apr 24 14:56:26 2022 -0400

    [SV 62175] Rework secondary expansion tests
    
    The hash function we use can yield different results on big- and
    little-endian systems which makes test output different.  Choose
    names to avoid this.
    
    * tests/scripts/features/patternrules: Choose portable target names.
    * tests/scripts/features/se_explicit: Ditto.
    * tests/scripts/features/se_implicit: Ditto.

-----------------------------------------------------------------------

Summary of changes:
 src/commands.c                      |   5 +-
 src/function.c                      |   9 +--
 src/hash.h                          |   8 +--
 src/implicit.c                      |  70 +++++++++++++-----
 src/main.c                          |   2 +-
 src/read.c                          |   5 +-
 src/rule.c                          |  13 +++-
 tests/scripts/features/patternrules | 138 ++++++++++++++++++++++++++++++++----
 tests/scripts/features/se_explicit  |  30 ++++----
 tests/scripts/features/se_implicit  |   8 +--
 10 files changed, 220 insertions(+), 68 deletions(-)


hooks/post-receive
-- 
make