FormatJson: Remove speculative comment
authorKevin Israel <pleasestand@live.com>
Fri, 25 Apr 2014 19:00:40 +0000 (15:00 -0400)
committerKevin Israel <pleasestand@live.com>
Fri, 25 Apr 2014 19:16:18 +0000 (15:16 -0400)
Follows-up bec7e8287c69. The comment "Can be removed once we require
PHP >= 5.4.28, 5.5.12, 5.6.0" relies on some assumptions that might
later prove to be incorrect:

* That the fix won't be reverted from any of those PHP versions
  (e.g. if deemed to break BC)

* That the bug will be fixed in PECL jsonc and jsond, as well as in
  HHVM

* That we don't need to support older versions of those once we
  require one of the mentioned PHP versions

Change-Id: I67034c561d54d37dee961ada8c9cf5ccfd113da1

includes/json/FormatJson.php

index eb5e632..e45dd3a 100644 (file)
@@ -155,7 +155,6 @@ class FormatJson {
 
                if ( $pretty !== false ) {
                        // Workaround for <https://bugs.php.net/bug.php?id=66021>
-                       // Can be removed once we require PHP >= 5.4.28, 5.5.12, 5.6.0
                        if ( $bug66021 ) {
                                $json = preg_replace( self::WS_CLEANUP_REGEX, '', $json );
                        }