Skip to content

Add returning reference support #123

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add returning reference support #123

wants to merge 2 commits into from

Conversation

andot
Copy link
Contributor

@andot andot commented Jul 12, 2014

The old implementation of PHP-CPP do not support returning reference, but now it can.

for example:

myNamespace.add("myFunction", myFunction, {}, true); myClass.method("myMethod", myMethod, {}, true);

Notice: Maybe this patch only works after patching #114.
andot added 2 commits July 13, 2014 06:43
@andot
Add returning reference support
05a91c7 
The old implementation of PHP-CPP do not support returning reference. but now it can.
@andot
Fixed reference count of return value.
26bcad5
@RogerWaters
Copy link

Hi,
is there any progress of this? It would be really nice if returning reference feature is available. Can you tell me if I can implement the magic function get like "public function &__get" with php-cpp?
Thanks for the nice wrapper.
RogerWaters

@andot tried your branch "returningref" and "master" but did not get returning reference feature to work. Is this because #114 ?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants
@andot @RogerWaters