Skip to content

Commit

Permalink
Merge pull request #543 from JuliaSymbolics/revert-542-return_tuples
Browse files Browse the repository at this point in the history
Revert "Included dispatch to toexpr that creates Tuples correctly"
  • Loading branch information
shashi authored Aug 20, 2023
2 parents e4812a3 + 9cc8fbd commit 8f9f5f1
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 6 deletions.
4 changes: 0 additions & 4 deletions src/code.jl
Original file line number Diff line number Diff line change
Expand Up @@ -188,10 +188,6 @@ function toexpr(O, st)
end
end

function toexpr(O::Tuple, st)
:(($(toexpr.(O, (st,))...),))
end

# Call elements of vector arguments by their name.
@matchable struct DestructuredArgs
elems
Expand Down
2 changes: 0 additions & 2 deletions test/code.jl
Original file line number Diff line number Diff line change
Expand Up @@ -183,8 +183,6 @@ nanmath_st.rewrites[:nanmath] = true
:(SparseVector(10, $(spvec.nzind), [a])))
test_repr(toexpr(MakeTuple((a, b, a+b))),
:((a,b,$(+)(a,b))))
test_repr(toexpr((a, b, a+b)),
:((a,b,$(+)(a,b))))

@test SpawnFetch{Multithreaded}([()->1,()->2],vcat)|>toexpr|>eval == [1,2]
@test @elapsed(SpawnFetch{Multithreaded}([:(()->sleep(2)),
Expand Down

0 comments on commit 8f9f5f1

Please sign in to comment.