Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve RecursiveIteratorIterator zend compatibility #1006

Closed
wants to merge 1 commit into from

Conversation

javer
Copy link
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
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
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