Skip to content

Commit 899b916

Browse files
committed
Merge declarations and assignments in php_dom_iterator_move_forward()
1 parent 7a3516c commit 899b916

File tree

1 file changed

+9
-14
lines changed

1 file changed

+9
-14
lines changed

ext/dom/dom_iterators.c

Lines changed: 9 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -175,29 +175,23 @@ static void php_dom_iterator_current_key(zend_object_iterator *iter, zval *key)
175175

176176
static void php_dom_iterator_move_forward(zend_object_iterator *iter) /* {{{ */
177177
{
178-
zval *object;
179-
xmlNodePtr curnode = NULL, basenode;
180-
dom_object *intern;
181-
dom_object *nnmap;
182-
dom_nnodemap_object *objmap;
183-
int previndex;
184-
HashTable *nodeht;
185-
zval *entry;
178+
xmlNodePtr curnode = NULL;
186179
bool do_curobj_undef = 1;
187180

188181
php_dom_iterator *iterator = (php_dom_iterator *)iter;
189182

190-
object = &iterator->intern.data;
191-
nnmap = Z_DOMOBJ_P(object);
192-
objmap = (dom_nnodemap_object *)nnmap->ptr;
183+
zval *object = &iterator->intern.data;
184+
dom_object *nnmap = Z_DOMOBJ_P(object);
185+
dom_nnodemap_object *objmap = nnmap->ptr;
193186

194-
intern = Z_DOMOBJ_P(&iterator->curobj);
187+
dom_object *intern = Z_DOMOBJ_P(&iterator->curobj);
195188

196189
if (intern != NULL && intern->ptr != NULL) {
197190
if (objmap->nodetype != XML_ENTITY_NODE &&
198191
objmap->nodetype != XML_NOTATION_NODE) {
199192
if (objmap->nodetype == DOM_NODESET) {
200-
nodeht = HASH_OF(&objmap->baseobj_zv);
193+
HashTable *nodeht = HASH_OF(&objmap->baseobj_zv);
194+
zval *entry;
201195
zend_hash_move_forward_ex(nodeht, &iterator->pos);
202196
if ((entry = zend_hash_get_current_data_ex(nodeht, &iterator->pos))) {
203197
zval_ptr_dtor(&iterator->curobj);
@@ -213,10 +207,11 @@ static void php_dom_iterator_move_forward(zend_object_iterator *iter) /* {{{ */
213207
} else {
214208
/* The collection is live, we nav the tree from the base object if we cannot
215209
* use the cache to restart from the last point. */
216-
basenode = dom_object_get_node(objmap->baseobj);
210+
xmlNodePtr basenode = dom_object_get_node(objmap->baseobj);
217211
if (UNEXPECTED(!basenode)) {
218212
goto err;
219213
}
214+
int previndex;
220215
if (php_dom_is_cache_tag_stale_from_node(&iterator->cache_tag, basenode)) {
221216
php_dom_mark_cache_tag_up_to_date_from_node(&iterator->cache_tag, basenode);
222217
previndex = 0;

0 commit comments

Comments
 (0)