Skip to content

Conversation

@NikoGrano
Copy link
Contributor

In case you use pthreads or something else related to threading you might encounter this issue. It happens only with functions and is fixable only by adding these checks. Another way to avoid this is to wrap functions as static into classes.

@NikoGrano
Copy link
Contributor Author

Related clue/stream-filter#23

@NikoGrano
Copy link
Contributor Author

Just a bumb. This PR has been also updated 🙃

@NikoGrano
Copy link
Contributor Author

Status?

@clue
Copy link
Owner

clue commented Mar 27, 2019

@Niko9911 Changes LGTM, can you squash this into a single commit like in reactphp/promise-timer#36? :shipit:

@NikoGrano
Copy link
Contributor Author

Made requested changes. Thank you. This can be merged now. @clue

🎖️

@clue
Copy link
Owner

clue commented Mar 27, 2019

Thanks for the quick update, now let's get this in! :shipit:

@clue clue merged commit b320957 into clue:master Mar 27, 2019
@NikoGrano NikoGrano deleted the patch-1 branch March 27, 2019 16:55
@NikoGrano
Copy link
Contributor Author

@clue Don't want be pinging all time, but could this be tagged?

@NikoGrano
Copy link
Contributor Author

@clue Can you tag this as 1.3.1?

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants