Skip to content

Conversation

sjkelly
Copy link
Contributor

@sjkelly sjkelly commented Nov 21, 2015

This undoes the deprecation introduced in #65 for call{T}(::Type{Node{T}}, x). AFAIK it wasn't an issue with syntax as much as the difficulty of finding the right Base.call (arggghh type invariance!).

@shashi
Copy link
Member

shashi commented Nov 23, 2015

Is there a way to keep Signal(T, value) working?

I think this is consistent with the Julia convention, e.g. Array(T, 10) etc.

@sjkelly
Copy link
Contributor Author

sjkelly commented Nov 23, 2015

Yes, I can do that. Since 0.4 Array{T}(10) has been supported with call, so it is helpful to have both I guess.

@sjkelly
Copy link
Contributor Author

sjkelly commented Nov 24, 2015

I lost some motivation rebasing for the Signal rename, so I just removed the deprecation and added constructor tests. This makes the patch much smaller.

@sjkelly
Copy link
Contributor Author

sjkelly commented Dec 16, 2015

Bump.

@sjkelly sjkelly closed this Aug 16, 2022
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