PhpLintTask performance improvement

[Elan Ruusamäe <glen@…>]

PhpLintTask calls out php -l, but php engine does not need any extension to check file for syntax errors, therefore a lot of time could be won if php-cli is told not to load php.ini and thus load and setup extensions:

so I propose to invoke as:

  php -n -l $FILE

my timing win is 10s -> 1s with just 69 files:

$ time phing phplint | grep -F '[phplint]' | wc -l
69
real    0m10.578s
user    0m7.846s
sys     0m2.530s

$ time phing phplint | grep -F '[phplint]' | wc -l
69
real    0m1.808s
user    0m0.927s
sys     0m0.683s

[Elan Ruusamäe <glen@…>]

proposed patch

[mrook]

Thanks!

[mrook]

(In [1310]) Fixes #797 - do not load php.ini file, increases php lint performance

[mrook]

(In [1312]) Refs #797 - set error_reporting to include E_DEPRECATED when deprecatedAsError is true

[Elan Ruusamäe <glen@…>]

Replying to mrook:

(In [1312]) Refs #797 - set error_reporting to include E_DEPRECATED when deprecatedAsError is true

i'm sure you as maintainer know what you're doing, but what about other error_level options that _may_ have been configured in global php.ini? perhaps just try to collect them and pass whole bitmask not just deprecated?

[mrook]

Replying to Elan Ruusamäe <glen@…>:

Replying to mrook:

(In [1312]) Refs #797 - set error_reporting to include E_DEPRECATED when deprecatedAsError is true

i'm sure you as maintainer know what you're doing, but what about other error_level options that _may_ have been configured in global php.ini? perhaps just try to collect them and pass whole bitmask not just deprecated?

The objective of your patch was to improve the performance by not loading the php.ini. This change is needed to facilitate detection of E_DEPRECATED errors.

[Elan Ruusamäe <glen@…>]

Replying to mrook:

The objective of your patch was to improve the performance by not loading the php.ini. This change is needed to facilitate detection of E_DEPRECATED errors.

I was kind of thinking inheritance of settings from already running Phing process (not each time php is about to invoked). Indeed those could had been already modified by some other options or Phing internal states. So if only deprecation is needed, all is fine :)