Skip to content

Commit

Permalink
[fix] Remove occurrences of E_STRICT
Browse files Browse the repository at this point in the history
Signed-off-by: Edward Z. Yang <[email protected]>
  • Loading branch information
ezyang committed Nov 10, 2024
1 parent cb56001 commit 7f7ea68
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 5 deletions.
2 changes: 1 addition & 1 deletion configdoc/generate.php
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
*/

if (version_compare(PHP_VERSION, '5.2', '<')) exit('PHP 5.2+ required.');
error_reporting(E_ALL | E_STRICT);
error_reporting(E_ALL);

// load dual-libraries
require_once dirname(__FILE__) . '/../extras/HTMLPurifierExtras.auto.php';
Expand Down
2 changes: 1 addition & 1 deletion tests/common.php
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ function test_autoload($class)
}

// after external libraries are loaded, turn on compile time errors
error_reporting(E_ALL | E_STRICT);
error_reporting(E_ALL);

// initialize extra HTML Purifier libraries
require '../extras/HTMLPurifierExtras.auto.php';
Expand Down
5 changes: 2 additions & 3 deletions tests/index.php
Original file line number Diff line number Diff line change
Expand Up @@ -23,9 +23,8 @@
* $test_files) do not have underscores in their names.
*/

// HTML Purifier runs error free on E_STRICT, so if code reports
// errors, we want to know about it.
error_reporting(E_ALL | E_STRICT);
// HTML Purifier runs error free.
error_reporting(E_ALL);

// Because we always want to know about errors, and because SimpleTest
// will notify us about them, logging the errors to stderr is
Expand Down

0 comments on commit 7f7ea68

Please sign in to comment.