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

Rename functions that conflict with debase's #447

Conversation

tzmfreedom
Copy link
Contributor

On my last pullrequest (#443), I fixed conflict with debase by renaming context initializer functions(e.g. Init_context() => Init_byebug_context()).

But I missed to fix other functions to conflict with debase's, for example Init_breakpoint(). So breakpoint isn't worked when using with debase.

I fixed it on this PR. On this PR, we can use byebug with debase completely.

Fixed point(rename) is the following

I think extern function name is unique globally as much as possible because C doesn't have namespace to function name.
For example, on msgpack-ruby(https://github.com/msgpack/msgpack-ruby), there is some functions that is prefixed with MessagePack or msgpack.

So ideally we may rename all extern function name by adding prefix(e.g. byebug_xxx()).
But I fixed only functions that conflict with debase now.
(I don't know the reason, some functions is the same as debase's.)

@tzmfreedom tzmfreedom force-pushed the prevent_conflict_to_debase branch 2 times, most recently from 29253c5 to 4116a0b Compare March 26, 2018 12:58
@deivid-rodriguez
Copy link
Owner

Thank you again @tzmfreedom! Looks good to me. I'll merge and release 10.0.2 with it as soon as I fixed CI configuration :)

@deivid-rodriguez
Copy link
Owner

@tzmfreedom Could you rebase this PR now? :)

@deivid-rodriguez
Copy link
Owner

Thanks so much for the patch. The reason for the name conflicts is that the C extension of byebug was initially forked from debase actually :)

I'll release 10.0.2 tomorrow with this fix 👍

@deivid-rodriguez deivid-rodriguez merged commit 71ec509 into deivid-rodriguez:master Mar 30, 2018
@tzmfreedom tzmfreedom deleted the prevent_conflict_to_debase branch March 30, 2018 04:14
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.

None yet

2 participants