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.

Sign up for GitHub

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

Jump to bottom

VM: allow void as a type argument #28945

Closed
floitschG opened this issue Mar 1, 2017 · 3 comments
Closed

VM: allow void as a type argument #28945

floitschG opened this issue Mar 1, 2017 · 3 comments
Assignees
@crelier
Labels
area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. closed-duplicate Closed in favor of an existing report

Comments

@floitschG
Copy link
Contributor

This is the VM issue for #27504.
@floitschG floitschG added the area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. label Mar 1, 2017
@floitschG floitschG added this to the 1.23 milestone Mar 1, 2017
@munificent munificent mentioned this issue Mar 1, 2017
Closed
5 tasks
@mit-mit mit-mit modified the milestones: 1.24, 1.23 Mar 15, 2017
@mit-mit
Copy link
Member

mit-mit commented Mar 15, 2017

Punting to 1.24 per dart-leads discussion.

@floitschG floitschG modified the milestones: 1.25, 1.24 May 9, 2017
@a-siva a-siva assigned a-siva and crelier and unassigned a-siva May 30, 2017
@crelier
Copy link
Contributor

crelier commented May 31, 2017 •
edited
Loading

It is not clear to me what feature is required here. It is easy to syntactically allow void as a type argument (see https://codereview.chromium.org/2907423003/), but I need to know how to handle void in type tests (non-strong mode and strong mode). So we need a spec or at least a set of dart tests setting some expectations. Thanks.

@eernstg
Copy link
Member

eernstg commented May 31, 2017

Right, you do of course need that, as do others---and I'll be writing such a spec (in the form of a standalone informal spec, as usual). We're GoogleServing today, but I'll get started writing it tomorrow.

@crelier crelier closed this as completed in bd39b94 Jun 5, 2017
@anders-sandholm anders-sandholm added the closed-duplicate Closed in favor of an existing report label Nov 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@crelier crelier

Labels
area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. closed-duplicate Closed in favor of an existing report
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@anders-sandholm @crelier @floitschG @a-siva @eernstg @mit-mit