Address CR comments

Signed-off-by: Bob Weinand <bobwei9@hotmail.com>

Author: bwoebi

Zend/zend_hash.c

@@ -2391,7 +2391,7 @@ static zend_always_inline uint32_t zend_array_dup_elements(HashTable *source, Ha
 			uint32_t target_idx = idx;
 
 			idx++; p++;
-			if (EXPECTED(!HT_HAS_ITERATORS(target))) {
+			if (!with_holes || EXPECTED(!HT_HAS_ITERATORS(target))) {
 				while (p != end) {
 					if (zend_array_dup_element(source, target, target_idx, p, q, 0, static_keys, with_holes)) {
 						if (source->nInternalPointer == idx) {

Zend/zend_types.h

@@ -533,7 +533,7 @@ typedef uint32_t HashPosition;
 typedef struct _HashTableIterator {
 	HashTable    *ht;
 	HashPosition  pos;
-	uint32_t      next_copy;
+	uint32_t      next_copy; // circular linked list via index into EG(ht_iterators)
 } HashTableIterator;
 
 struct _zend_object {