ext/session: refactor session_reset()

By making the underlying function return a bool and not do duplicate checks

Author: Girgias

ext/session/session.c

@@ -1747,13 +1747,13 @@ static bool php_session_abort(void)
 	return false;
 }
 
-static zend_result php_session_reset(void)
+static bool php_session_reset(void)
 {
 	if (PS(session_status) == php_session_active
 		&& php_session_initialize() == SUCCESS) {
-		return SUCCESS;
+		return true;
 	}
-	return FAILURE;
+	return false;
 }
 
 
@@ -2748,11 +2748,7 @@ PHP_FUNCTION(session_reset)
 		RETURN_THROWS();
 	}
 
-	if (PS(session_status) != php_session_active) {
-		RETURN_FALSE;
-	}
-	php_session_reset();
-	RETURN_TRUE;
+	RETURN_BOOL(php_session_reset());
 }
 
 PHP_FUNCTION(session_status)