[openssl/openssl] 2d2c2b: test/recipes/05-test_rand.t: replace 'and' with '&&'

Richard Levitte noreply at github.com
Wed Oct 11 12:21:27 UTC 2023


  Branch: refs/heads/openssl-3.1
  Home:   https://github.com/openssl/openssl
  Commit: 2d2c2beda97f8e0e9c91c0ee245c3ace557c0437
      https://github.com/openssl/openssl/commit/2d2c2beda97f8e0e9c91c0ee245c3ace557c0437
  Author: Richard Levitte <levitte at openssl.org>
  Date:   2023-10-11 (Wed, 11 Oct 2023)

  Changed paths:
    M test/recipes/05-test_rand.t

  Log Message:
  -----------
  test/recipes/05-test_rand.t: replace 'and' with '&&'

The lower priority 'and' seems to have some "interesting" interactions with
function argument parsing in some perl versions (presumably because 'and' is
lower priority than the comma).

For the lines that are changed here, perl v5.20.1 says this:

    Useless use of string eq in void context at [.test.recipes]05-test_rand.t line 33.
    Useless use of numeric eq (==) in void context at [.test.recipes]05-test_rand.t line 39.

Replacing 'and' with '&&' in these two cases fixes the problem.

Replacing

Reviewed-by: Hugo Landau <hlandau at openssl.org>
Reviewed-by: Matt Caswell <matt at openssl.org>
Reviewed-by: Paul Dale <pauli at openssl.org>
Reviewed-by: Tomas Mraz <tomas at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/22331)

(cherry picked from commit 715242b1ca2b7267a70fb13c3544a84b947a6e81)




More information about the openssl-commits mailing list