On ReflectionParameter::canBePassedByValue()

Hi. As you may or may not know, the "zend_bool" arg_info->pass_by_reference, may take three values:

785 #define ZEND_SEND_BY_VAL     0
786 #define ZEND_SEND_BY_REF     1

787 #define ZEND_SEND_PREFER_REF 2 The last one is used one the function can either take a value or reference; see e.g. array_multisort. ReflectionParameter has only only method, with an actual boolean return to test this, isPassedByReference(). This returns true for ZEND_SEND_BY_REF or ZEND_SEND_PREFER_REF, so there's no way to distinguish between the two. To solve this, I added to trunk ReflectionParameter::canBePassedByValue(), which tests for ZEND_SEND_BY_VAL or ZEND_SEND_PREFER_REF. Concerns have been raised over the naming of the method and over whether having three methods for three values would be more appropriate (with a minor BC break, changing the behavior of isPassedByReference() to return true only with ZEND_SEND_BY_REF). Any comments on this issue are appreciated. Thanks -- Gustavo Lopes