Edit report at https://bugs.php.net/bug.php?id=74082&edit=1

 ID:                 74082
 Updated by:         c...@php.net
 Reported by:        edward dot vielmetti at gmail dot com
 Summary:            strncmp_variation6 test fails on AArch64
-Status:             Open
+Status:             Closed
 Type:               Bug
 Package:            Testing related
 Operating System:   Ubuntu 16.04 LTS on aarch64
 PHP Version:        7.1.1
-Assigned To:        
+Assigned To:        cmb
 Block user comment: N
 Private report:     N

 New Comment:

As discussed on PR #3689[1], the pointless test has been dropped
altogether[2].

[1] <https://github.com/php/php-src/pull/3689>
[2] 
<http://git.php.net/?p=php-src.git;a=commit;h=ea1ea33bd5d967e9c04046c1af869e4e8b53116e>


Previous Comments:
------------------------------------------------------------------------
[2018-07-16 13:12:08] c...@php.net

This test[1] confuses binary strings and binary numbers, and
should better be rewritten or dropped, anyway.

Furthermore, and this is might be the reason for the reported test
failure, the test assumes that strncmp() would return exactly 1,
if the first string is greater than the second, but strncmp() only
guarantees that a positive integer is returned in this case.

[1] 
<https://github.com/php/php-src/blob/php-7.1.1/ext/standard/tests/strings/strncmp_variation6.phpt>

------------------------------------------------------------------------
[2017-02-10 21:42:05] edward dot vielmetti at gmail dot com

root@docker-build-test:/mnt/src/php/php-7.1.1/ext/standard/tests/strings# more 
strncmp*diff
774+ int(128)
774- int(1)

------------------------------------------------------------------------
[2017-02-10 21:19:41] ni...@php.net

What's the diff? (ext/standard/tests/strings/strncmp_variation6.diff)

------------------------------------------------------------------------
[2017-02-10 21:01:20] edward dot vielmetti at gmail dot com

Description:
------------
FAILED TEST SUMMARY
---------------------------------------------------------------------
Test strncmp() function : usage variations - binary safe(binary values) 
[ext/standard/tests/strings/strncmp_variation6.phpt]

"make test" fails this one test on aarch64 (ARMv8, arm64) on Packet 2A 96-core 
system.



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



--
Edit this bug report at https://bugs.php.net/bug.php?id=74082&edit=1

Reply via email to