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

code_lowered error message #4023

Closed
mlubin opened this issue Aug 12, 2013 · 5 comments
Closed

code_lowered error message #4023

mlubin opened this issue Aug 12, 2013 · 5 comments

Comments

@mlubin
Copy link
Member

mlubin commented Aug 12, 2013

This could give a better error message:

julia> code_lowered(map,(Int,))
ERROR: BoundsError()
 in code_lowered at reflection.jl:75
@JeffBezanson
Copy link
Sponsor Member

This reminds me of a reason I don't like these functions that I had
forgotten about. These were originally for internal, debugging purposes,
but now we have to put nice interfaces on them and maintain them.
On Aug 11, 2013 10:19 PM, "Miles Lubin" [email protected] wrote:

This could give a better error message:

julia> code_lowered(map,(Int,))
ERROR: BoundsError()
in code_lowered at reflection.jl:75


Reply to this email directly or view it on GitHubhttps://github.com//issues/4023
.

@mlubin
Copy link
Member Author

mlubin commented Aug 12, 2013

Unfortunately they're too useful :)

@vtjnash
Copy link
Sponsor Member

vtjnash commented Aug 12, 2013

I believe my inlining pull request #3796 includes a patch for this as part of another bug fix.

@JeffBezanson
Copy link
Sponsor Member

As always, it would be good to have applicable bug fixes on master.
On Aug 12, 2013 1:36 AM, "Jameson Nash" [email protected] wrote:

I believe my inlining pull request #3796https://github.com/JuliaLang/julia/issues/3796includes a patch for this as part of another bug fix.


Reply to this email directly or view it on GitHubhttps://github.com//issues/4023#issuecomment-22474091
.

@vtjnash
Copy link
Sponsor Member

vtjnash commented Aug 12, 2013

@JeffBezanson Can I apply the changes to _methods and finfer from d436341 (also fixes #3167 and #3400 -- the other parts of that commit are unrelated)?

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

No branches or pull requests

3 participants