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

Fix goto/label in macros invoked from another module. #7882

Merged
merged 1 commit into from
Aug 7, 2014

Conversation

dcjones
Copy link
Contributor

@dcjones dcjones commented Aug 7, 2014

This sort of thing previously incorrectly threw an error

module A
    macro some_macro()
        quote
            function $(esc(:foo))
                @label a
                @goto a
            end
        end
    end
end

A.@some_macro

It would try to rewrite @label a with @label A.a, this commit adds an special case to avoid that.

JeffBezanson added a commit that referenced this pull request Aug 7, 2014
Fix goto/label in macros invoked from another module.
@JeffBezanson JeffBezanson merged commit b21c386 into master Aug 7, 2014
@dcjones dcjones deleted the dcj/gotobug branch August 10, 2014 19:19
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