Skip to content

Improve RecursiveIteratorIterator zend compatibility#1006

Closed
javer wants to merge 1 commit intofacebook:masterfrom
javer:spl-recursiveiteratoriterator
Closed

Improve RecursiveIteratorIterator zend compatibility#1006
javer wants to merge 1 commit intofacebook:masterfrom
javer:spl-recursiveiteratoriterator

Conversation

@javer
Copy link
Copy Markdown
Contributor

@javer javer commented Sep 1, 2013

Added methods and callbacks: beginChildren, beginIteration, callGetChildren, callHasChildren, endChildren, endIteration, getDepth, getMaxDepth, getSubIterator, nextElement, setMaxDepth.

Added methods and callbacks: beginChildren, beginIteration,
callGetChildren, callHasChildren, endChildren, endIteration, getDepth,
getMaxDepth, getSubIterator, nextElement, setMaxDepth.
@scannell
Copy link
Copy Markdown
Contributor

scannell commented Sep 4, 2013

Cool, thanks @javer. I'm going to leave the 070 test out for now and should merge the rest shortly.

@scannell
Copy link
Copy Markdown
Contributor

scannell commented Sep 6, 2013

Due to a technical issue unrelated to this diff, it's going to be a few days before this lands. Sorry about the delay.

@sgolemon sgolemon closed this in f4eaa42 Sep 9, 2013
facebook-github-bot pushed a commit that referenced this pull request Oct 4, 2024
Summary: [Fix an upstream bug](mozilla/cbindgen#1006) and [add ability to put arbitrary prefix on field names](mozilla/cbindgen#1007).

Reviewed By: jasonwhite

Differential Revision: D63878067

fbshipit-source-id: 16a35ed1fa2adc4c9211fc6f6332f0be7dbf4f56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants