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

Add EComplexType expression variant #6913

Closed
wants to merge 1 commit into
base: development
from

Conversation

Projects
None yet
4 participants
@bendmorris
Copy link
Contributor

bendmorris commented Mar 18, 2018

Proposed here, this is a way to specify types that can't be parsed as identifiers, e.g. parameterized types:

var myType = :Array<MyClass>;

@:meta(:a.b.SomeType<P>) public function myFunction() {}

@bendmorris bendmorris force-pushed the bendmorris:complex-type-literal branch from f571053 to 30ba658 Mar 18, 2018

@RealyUniqueName RealyUniqueName requested a review from ncannasse Mar 18, 2018

@bendmorris bendmorris referenced this pull request Mar 18, 2018

Closed

Complex type expressions #44

@skial skial referenced this pull request Mar 22, 2018

Closed

Haxe Roundup 424 #490

1 of 1 task complete
@Simn

This comment has been minimized.

Copy link
Member

Simn commented Apr 17, 2018

@nadako Do you remember what grammar problem we ran into with : a while back? I think it had to do with case.

@nadako

This comment has been minimized.

Copy link
Member

nadako commented Apr 17, 2018

it was #6433, and this is also problematic because macro <colon> <something> is already used for reifying complex types...

@Simn

This comment has been minimized.

Copy link
Member

Simn commented Apr 17, 2018

Right, I think case is fine here, but the macro : CT situation means that you couldn't reify EComplexType. That's a bit annoying...

@back2dos

This comment has been minimized.

Copy link
Member

back2dos commented Apr 17, 2018

macro : CT situation means that you couldn't reify EComplexType. That's a bit annoying...

I'd argue that macro (:CT) is close enough, if you really need it ;)

@Simn Simn added this to the Design milestone Apr 20, 2018

@Simn Simn self-assigned this Apr 20, 2018

@Simn

This comment has been minimized.

Copy link
Member

Simn commented Apr 30, 2018

Given that this doesn't merge and HaxeFoundation/haxe-evolution#44 is unlikely to be accepted, I'll go ahead and close this PR for now.

@Simn Simn closed this Apr 30, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment