Merge pull request #2 from Riimu/php7

Ensure that tests pass on php7 & hhvm

Author: Riimu

.travis.yml

@@ -5,6 +5,8 @@ php:
   - 5.6
   - 5.5
   - 5.4
+  - 7.0
+  - hhvm
 
 cache:
   directories:
@@ -14,8 +16,8 @@ before_install:
   - composer self-update
 
 install:
-  - composer require --no-update --no-interaction "phpunit/phpunit:4.*" "squizlabs/php_codesniffer:2.*" "fabpot/php-cs-fixer:1.*"
-  - travis_retry composer install --no-interaction
+  - composer require --no-update --no-interaction "phpunit/phpunit:*" "squizlabs/php_codesniffer:*" "fabpot/php-cs-fixer:*"
+  - travis_retry composer install --no-interaction --prefer-source
   - travis_retry wget https://scrutinizer-ci.com/ocular.phar
 
 script:
@@ -24,4 +26,4 @@ script:
   - vendor/bin/php-cs-fixer fix --dry-run --diff
 
 after_script:
-  - php ocular.phar code-coverage:upload --format=php-clover coverage.clover
+  - if [ -f coverage.clover ]; then php ocular.phar code-coverage:upload --format=php-clover coverage.clover; fi

README.md

@@ -243,13 +243,23 @@ apply to following calls.
 
 ## Known Issues ##
 
-If you are running this library on a PHP version earlier than 5.4.5, you may
-receive the following error, if you are trying to encode a recursive array:
+### Recursive arrays on PHP < 5.4.5 ###
+
+If you try to encode recursive arrays on PHP versions earlier than 5.4.5, you
+may encounter the following error (due to the way array comparisons work
+internally in PHP):
 
 `Fatal error:  Nesting level too deep - recursive dependency?`
 
-In order to fix this, it is recommended to disable `recursion.detect` and set
-a max value for `recursion.max` if you expect to encounter recursive arrays.
+In order to fix this, you should disable the recursive array detection by
+setting the option `recursion.detect` to false and set a value for the option
+`recursion.max` to prevent recursive arrays from causing an infinite loop.
+
+### Float precision on HHVM ###
+
+Note that HHVM does not support the ini setting `serialize_precision`. Thus,
+the use of the value `false` for the option `float.precision` is not supported
+on HHVM.
 
 ## Credits ##
 

tests/tests/EncodingTest.php

@@ -93,16 +93,17 @@ public function testFloatExponents()
         $this->assertEncode('-1.0E-32', -1.0e-32);
     }
 
-    public function testPHPDefaultPrecision()
+    public function testUsingIniPrecision()
     {
-        $float = 1.12345678901234567890;
+        if (defined('HHVM_VERSION')) {
+            $this->markTestSkipped();
+        }
 
-        $cast = ini_set('precision', 17);
-        $serialize = ini_set('serialize_precision', 17);
+        $float = 1.1234567890123456;
+        $serialize = ini_set('serialize_precision', 13);
 
-        $this->assertEncode(var_export($float, true), $float, ['float.precision' => false]);
+        $this->assertEncode('1.123456789012', $float, ['float.precision' => false]);
 
-        ini_set('precision', $cast);
         ini_set('serialize_precision', $serialize);
     }