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

cconvert stack overflow with invalid input type #268

Open
johnnychen94 opened this issue Nov 2, 2021 · 0 comments
Open

cconvert stack overflow with invalid input type #268

johnnychen94 opened this issue Nov 2, 2021 · 0 comments
Labels

Comments

@johnnychen94
Copy link
Member

It seems that our convert dispatch workflow is a little bit fragile 😆

julia> Gray(1.0+0.0im)
ERROR: StackOverflowError:
Stacktrace:
     [1] cconvert(#unused#::Type{Gray{N0f8}}, c::Complex{Int64})
       @ ColorTypes ~/.julia/packages/ColorTypes/6m8P7/src/conversions.jl:76
     [2] convert(#unused#::Type{Gray{N0f8}}, c::Complex{Int64})
       @ ColorTypes ~/.julia/packages/ColorTypes/6m8P7/src/conversions.jl:74
     [3] Gray{N0f8}(x::Complex{Int64})
       @ ColorTypes ~/.julia/packages/ColorTypes/6m8P7/src/types.jl:465
     [4] _convert(#unused#::Type{Gray{N0f8}}, #unused#::Type{Gray}, #unused#::Type{Gray}, c::Complex{Int64})
       @ ColorTypes ~/.julia/packages/ColorTypes/6m8P7/src/conversions.jl:96
--- the last 4 lines are repeated 19994 more times ---
 [79981] cconvert(#unused#::Type{Gray{N0f8}}, c::Complex{Int64})
       @ ColorTypes ~/.julia/packages/ColorTypes/6m8P7/src/conversions.jl:76
 [79982] convert(#unused#::Type{Gray{N0f8}}, c::Complex{Int64})
       @ ColorTypes ~/.julia/packages/ColorTypes/6m8P7/src/conversions.jl:74
 [79983] Gray{N0f8}(x::Complex{Int64})
       @ ColorTypes ~/.julia/packages/ColorTypes/6m8P7/src/types.jl:465
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant