Simplify IsPacked

Author: alexandre-daubois

docs/extensions.md

@@ -212,7 +212,7 @@ func process_data_packed(arr *C.zend_array) unsafe.Pointer {
 - `frankenphp.GoAssociativeArray(arr unsafe.Pointer, ordered bool) frankenphp.AssociativeArray` - Convert a PHP array to an ordered Go `AssociativeArray` (map with order)
 - `frankenphp.GoMap(arr unsafe.Pointer) map[string]any` - Convert a PHP array to an unordered Go map
 - `frankenphp.GoPackedArray(arr unsafe.Pointer) []any` - Convert a PHP array to a Go slice
-- `frankenphp.IsPacked(zval unsafe.Pointer) bool` - Check if a PHP array is packed (indexed only) or associative (key-value pairs)
+- `frankenphp.IsPacked(zval *C.zend_array) bool` - Check if a PHP array is packed (indexed only) or associative (key-value pairs)
 
 ### Working with Callables
 

types.go

@@ -416,24 +416,14 @@ func createNewArray(size uint32) *C.zend_array {
 	return (*C.zend_array)(unsafe.Pointer(arr))
 }
 
-// IsPacked determines if the given zval pointer represents a packed array (list).
-// Returns false if the zval is nil, not an array, or not packed.
-func IsPacked(zval unsafe.Pointer) bool {
-	if zval == nil {
-		return false
-	}
-
-	v, err := extractZvalValue((*C.zval)(zval), C.IS_ARRAY)
-	if err != nil {
-		return false
-	}
-
-	ht := (*C.HashTable)(v)
-	if ht == nil {
+// IsPacked determines if the given zend_array is a packed array (list).
+// Returns false if the array is nil or not packed.
+func IsPacked(arr unsafe.Pointer) bool {
+	if arr == nil {
 		return false
 	}
 
-	return htIsPacked(ht)
+	return htIsPacked((*C.zend_array)(arr))
 }
 
 // htIsPacked checks if a zend_array is a list (packed) or hashmap (not packed).