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

Old type parameter syntax deprecation loses information #23010

Closed
ararslan opened this issue Jul 28, 2017 · 0 comments
Closed

Old type parameter syntax deprecation loses information #23010

ararslan opened this issue Jul 28, 2017 · 0 comments
Assignees
Labels
bug Indicates an unexpected problem or unintended behavior deprecation This change introduces or involves a deprecation

Comments

@ararslan
Copy link
Member

The following message came from JSON.jl:

WARNING: deprecated syntax "parametric method syntax parse{T <: Associative}((parameters (kw (:: dicttype (curly Type T)) (curly Dict String Any))),io::IO) around /Users/alex/.julia/v0.7/JSON/src/Parser.jl:386".
Use "parse(#self#::Core.Typeof(parse),io::IO) where T" instead.

Note how the parsed representation leaks into the method signature, and how the initial use has the constraint T<:Associative, whereas the suggested replacement has no bound on T, nor does it list the keyword arguments present in the original signature.

This was introduced by #22834.

@ararslan ararslan added bug Indicates an unexpected problem or unintended behavior deprecation This change introduces or involves a deprecation labels Jul 28, 2017
@JeffBezanson JeffBezanson self-assigned this Jul 28, 2017
JeffBezanson added a commit that referenced this issue Jul 29, 2017
fix #23010, method parameter deprecation warning fixes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Indicates an unexpected problem or unintended behavior deprecation This change introduces or involves a deprecation
Projects
None yet
Development

No branches or pull requests

2 participants