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

union of tuples is broken #25338

Closed
ajkeller34 opened this issue Dec 31, 2017 · 1 comment
Closed

union of tuples is broken #25338

ajkeller34 opened this issue Dec 31, 2017 · 1 comment
Labels
collections Data structures holding multiple items, e.g. sets regression Regression in behavior compared to a previous version

Comments

@ajkeller34
Copy link
Contributor

On 0.6.2:

julia> union((1,2))
2-element Array{Int64,1}:
 1
 2

0.7.0-DEV.3222:

julia> union((1,2))
ERROR: MethodError: no method matching emptymutable(::Tuple{Int64,Int64}, ::Type{Int64})
...

You can try to take the union of multiple tuples (e.g. union((1,2), (3,))) and that doesn't work either. I haven't bisected to be sure, but this broke quite recently so I think #23528 is responsible.

cc @rfourquet

@ajkeller34 ajkeller34 changed the title union of Tuples is broken Union of Tuples is broken Dec 31, 2017
@ararslan ararslan added collections Data structures holding multiple items, e.g. sets regression Regression in behavior compared to a previous version labels Dec 31, 2017
@JeffBezanson JeffBezanson changed the title Union of Tuples is broken union of tuples is broken Dec 31, 2017
@rfourquet
Copy link
Member

Thanks for reporting! emptymutable was meant to have a default fall-back, but I forgot about it and no test caught this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
collections Data structures holding multiple items, e.g. sets regression Regression in behavior compared to a previous version
Projects
None yet
Development

No branches or pull requests

3 participants