merged branch nikic/improveIsDefinedPerformanceInNonStrictMode (PR #381)
authorFabien Potencier <fabien.potencier@gmail.com>
Tue, 12 Jul 2011 07:49:44 +0000 (09:49 +0200)
committerFabien Potencier <fabien.potencier@gmail.com>
Tue, 12 Jul 2011 07:49:44 +0000 (09:49 +0200)
commit90d99a7737270b2720b8ba88a9ad70dedade9cd5
treee424202218da2ee901e4770a2cc72ceca295125d
parent0ef96f6b1ddd2b395ab6476482fb4461e4036ec2
parente4b8371ab412b63a7472c36aa74c840ad4f142bc
merged branch nikic/improveIsDefinedPerformanceInNonStrictMode (PR #381)

Commits
-------

e4b8371 Improve performance of is defined in non strict mode

Discussion
----------

Improve performance of is defined in non strict mode

Addresses issue #380: This gives a good performance improvement on defined tests in non-strict code. About 2x.

Reasoning: It doesn't make sense to propagate the `is_defined_test` flag into deeper levels in non-strict mode, because all getAttr functions and name accesses will just return `null` if it doesn't exist and `null` is just as good as `false`.

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

by nikic at 2011/07/02 03:22:53 -0700

Though, do not merge yet please. Changes behavior concerning methods.

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

by nikic at 2011/07/02 13:34:32 -0700

Hm, I really don't know what I was thinking when I wrote my last comment. There shouldn't be any behavior changes introduced by that change (and tests pass).