From: Antony Dovgal <tony2001@php.net>
Date: Wed, 6 Jun 2007 11:57:36 +0000 (+0000)
Subject: MFH: minor optimization - make sure the objects are not the same instance before... 
X-Git-Tag: php-5.2.4RC1~411
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=e029945a02735f83391c60d0f42db75b0c900373;p=php

MFH: minor optimization - make sure the objects are not the same instance before we start comparing their properties
---

diff --git a/Zend/zend_operators.c b/Zend/zend_operators.c
index 814e07b9e3..cf46c513c1 100644
--- a/Zend/zend_operators.c
+++ b/Zend/zend_operators.c
@@ -1402,6 +1402,11 @@ ZEND_API int compare_function(zval *result, zval *op1, zval *op2 TSRMLS_DC)
 
 	/* If both are objects sharing the same comparision handler then use is */
 	if (eq_comp) {
+		if (Z_OBJ_HANDLE_P(op1) == Z_OBJ_HANDLE_P(op2)) {
+			/* object handles are identical, apprently this is the same object */
+			ZVAL_LONG(result, 0);
+			COMPARE_RETURN_AND_FREE(SUCCESS);
+		}
 		ZVAL_LONG(result, Z_OBJ_HT_P(op1)->compare_objects(op1, op2 TSRMLS_CC));
 		COMPARE_RETURN_AND_FREE(SUCCESS);
 	}