Skip to content
This repository was archived by the owner on Nov 10, 2017. It is now read-only.

Make extension work on php7#18

Open
sunkan wants to merge 1 commit intojedisct1:masterfrom
sunkan:php7
Open

Make extension work on php7#18
sunkan wants to merge 1 commit intojedisct1:masterfrom
sunkan:php7

Conversation

@sunkan
Copy link
Copy Markdown

@sunkan sunkan commented Jun 23, 2016

Update php extension to php7 api

@jedisct1
Copy link
Copy Markdown
Owner

Does it still work on php5 ?

@sunkan
Copy link
Copy Markdown
Author

sunkan commented Jun 23, 2016

No.

So would recommend putting it in a separate branch.

@jedisct1
Copy link
Copy Markdown
Owner

Having the same code work on both would be way better.

Maybe you can use macros similar to these: https://github.com/jedisct1/libsodium-php/blob/master/compat.h

@sunkan
Copy link
Copy Markdown
Author

sunkan commented Jun 23, 2016

That looks good I can implement that.

But that will have to wait till next week.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants