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

Throw genuine ErrorException. #11156

Closed
wants to merge 1 commit into from
Closed

Conversation

mschauer
Copy link
Contributor

@mschauer mschauer commented May 6, 2015

Compare the result of

julia> try throw("foobar"); catch e; dump(e);end
ASCIIString "foobar"

julia> try error("foobar"); catch e; dump(e);end
ErrorException 
  msg: ASCIIString "foobar"

See also #11146

@tkelman
Copy link
Contributor

tkelman commented May 6, 2015

would some existing more specific exception type be more appropriate for this?

@jakebolewski
Copy link
Member

@mschauer great! Could you have this throw an argument error and make the error message a bit more descriptive?

@mschauer
Copy link
Contributor Author

mschauer commented May 6, 2015

Yes, good idea.

@mschauer
Copy link
Contributor Author

mschauer commented May 6, 2015

Ah, updated master and saw that this is already fixed. d9a9cc4

@mschauer mschauer closed this May 6, 2015
@mschauer mschauer deleted the errorex branch May 6, 2015 14:08
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

3 participants