ext/intl: c++ memory management application to timezone class internals.

Author: devnexen

ext/intl/timezone/timezone_class.cpp

@@ -133,7 +133,7 @@ U_CFUNC TimeZone *timezone_process_timezone_argument(zval *zv_timezone,
 													 const char *func)
 {
 	zval		local_zv_tz;
-	TimeZone	*timeZone;
+	std::unique_ptr<TimeZone>	timeZone;
 
 	if (zv_timezone == NULL || Z_TYPE_P(zv_timezone) == IS_NULL) {
 		timelib_tzinfo *tzinfo = get_timezone_info();
@@ -153,12 +153,13 @@ U_CFUNC TimeZone *timezone_process_timezone_argument(zval *zv_timezone,
 			zval_ptr_dtor_str(&local_zv_tz);
 			return NULL;
 		}
-		timeZone = to->utimezone->clone();
-		if (UNEXPECTED(timeZone == NULL)) {
+		auto tmp = to->utimezone->clone();
+		if (UNEXPECTED(tmp == NULL)) {
 			zend_throw_error(IntlException_ce_ptr, "%s: could not clone TimeZone", func);
 			zval_ptr_dtor_str(&local_zv_tz);
 			return NULL;
 		}
+		timeZone = std::unique_ptr<TimeZone>(tmp);
 	} else if (Z_TYPE_P(zv_timezone) == IS_OBJECT &&
 			instanceof_function(Z_OBJCE_P(zv_timezone), php_date_get_timezone_ce())) {
 
@@ -181,24 +182,25 @@ U_CFUNC TimeZone *timezone_process_timezone_argument(zval *zv_timezone,
 			zval_ptr_dtor_str(&local_zv_tz);
 			return NULL;
 		}
-		timeZone = TimeZone::createTimeZone(id);
-		if (UNEXPECTED(timeZone == NULL)) {
+		auto tmp = TimeZone::createTimeZone(id);
+		if (UNEXPECTED(tmp == NULL)) {
 			zend_throw_error(IntlException_ce_ptr, "%s: Could not create time zone", func);
 			zval_ptr_dtor_str(&local_zv_tz);
 			return NULL;
 		}
+		timeZone = std::unique_ptr<TimeZone>(tmp);
 		if (*timeZone == TimeZone::getUnknown()) {
 			zend_throw_error(IntlException_ce_ptr, "%s: No such time zone: '%s'",
 				func, Z_STRVAL_P(zv_timezone));
 			zval_ptr_dtor_str(&local_zv_tz);
-			delete timeZone;
 			return NULL;
 		}
 	}
 
 	zval_ptr_dtor_str(&local_zv_tz);
 
-	return timeZone;
+	// well, this is included by the centralized C intl part so the "smart" part can't go further
+	return timeZone.release();
 }
 /* }}} */
 

ext/intl/timezone/timezone_class.h

@@ -36,6 +36,8 @@ typedef struct {
 	intl_error		err;
 
 	// ICU TimeZone
+	// TODO?: a direct change isn't possible due to C inclusion (also it s a const)
+	// but see later it can be made possible through different ICU class usages
 	const TimeZone	*utimezone;
 
 	//whether to delete the timezone on object free