You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The problem is that this encourages (forces?) users to implement methods like Base.tryparse(::Type{Vector{Float64}}, x::String), which is piracy.
Now, most likely, this will not cause issues, since users will only implement them if they find the methods to not exist in Base. However
It makes depdendents of Comonicon risky to use together, since different packages may define the same tryparse methods
These methods may be implemented in future versions of Julia
A straightforward solution would be to have a distinct function like Comonicon.try_parse_arg, which has a default definition try_parse_arg(x::Type{T}, s::AbstractString) where T = tryparse(T, s).
This should not cause breakage.
The text was updated successfully, but these errors were encountered:
The problem is that this encourages (forces?) users to implement methods like
Base.tryparse(::Type{Vector{Float64}}, x::String)
, which is piracy.Now, most likely, this will not cause issues, since users will only implement them if they find the methods to not exist in Base. However
tryparse
methodsA straightforward solution would be to have a distinct function like
Comonicon.try_parse_arg
, which has a default definitiontry_parse_arg(x::Type{T}, s::AbstractString) where T = tryparse(T, s)
.This should not cause breakage.
The text was updated successfully, but these errors were encountered: