Skip to content

Conversation

@kmsquire
Copy link
Member

@kmsquire kmsquire commented Jul 2, 2015

@sbromberger
Copy link
Contributor

@kmsquire thank you so much - this is awesome (and embarrassing since my code looked nothing like this - just shows how bad I am at metaprogramming).

@kmsquire
Copy link
Member Author

kmsquire commented Jul 2, 2015

:-)

See if you can understand it so that you can come up with something next time.

As a hint, I use a lot of dump(:(Vector{Int}()), 10) and macroexpand(:(@compat Vector{Int}())), in both v0.3 and v0.4).

I'll merge as soon as ci passes.

@sbromberger
Copy link
Contributor

Thanks. Please tag as well so I can REQUIRE it.

kmsquire added a commit that referenced this pull request Jul 2, 2015
at-compat for Vector{T}(), Vector{T}(n), Array{T}  (fixes #105)
@kmsquire kmsquire merged commit d12d6fb into master Jul 2, 2015
@kmsquire kmsquire deleted the kms/curly_vector_array branch July 2, 2015 04:29
martinholters pushed a commit to martinholters/Compat.jl that referenced this pull request Jul 13, 2016
Bug fix, and clean up of NA iterators
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants