Skip to content

Implement Binary Search from py2exe. #74

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

Closed
wants to merge 1 commit into from
Closed

Implement Binary Search from py2exe. #74

wants to merge 1 commit into from

Conversation

AraHaan
Copy link

@AraHaan AraHaan commented Oct 16, 2017

py2exe uses v0.0.3 modified to add binary search to load dll's (like pythonxx.dll) with a ton of exports.

Let me know if I need to update something to fix problems with this, I tried with a mere copy paste from the version of this that py2exe ships with. I would like this included so that way py2exe dont have to ship an old (patched) special version of the code that could easily have bugs that the current version on this git on branch master patches.So instead I made this Pull request to achieve such. Not to mention what if an in general user (non python or py2exe user needs this binary search that the original py2exe creator patched into this. Either way, I see this more of a useful thing to have in the mainline MemoryModule.

py2exe uses v0.0.3 modified to add binary search to load dll's (like pythonxx.dll) with a ton of exports.
fancycode added a commit that referenced this pull request Nov 2, 2017
Code based on #74 which contained code from py2exe.
@fancycode
Copy link
Owner

Implemented in #76, thanks for the heads up.

@fancycode fancycode closed this Nov 2, 2017
@AraHaan
Copy link
Author

AraHaan commented Nov 3, 2017

Thanks. And sorry for any code I made that did not work initially, I just cant hardly read C code that well as well it becomes all over the place sometimes. I should really consider learning how to read C so that way ceval.c in cpython is no so cancerous for me to even try to understand what it does.

@AraHaan AraHaan deleted the patch-1 branch November 3, 2017 06:03
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