Skip to content

class Null should be declared explicitly in the core library #12382

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

Closed
gbracha opened this issue Aug 12, 2013 · 5 comments
Closed

class Null should be declared explicitly in the core library #12382

gbracha opened this issue Aug 12, 2013 · 5 comments
Labels
area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. web-dart2js

Comments

@gbracha
Copy link
Contributor

gbracha commented Aug 12, 2013

Null is a real class and should have a declaration just like Object or anything else.

@munificent
Copy link
Member

With Futures, you often have a future that completes to a side-effect and no meaningful value. In that case, it always completes to null. If we have a public Null class, should we start using Future<Null> to indicate that?

@lrhn
Copy link
Member

lrhn commented Aug 12, 2013

In Dart2JS it already has.
Is this requesting that the class be public.

@gbracha
Copy link
Contributor Author

gbracha commented Aug 12, 2013

Re comment 2: yes, the class is named Null, so it cannot be private.

@rmacnak-google
Copy link
Contributor

Marked this as being blocked by #12364.

@lrhn
Copy link
Member

lrhn commented Aug 16, 2013

Added Fixed label.

@gbracha gbracha added Type-Defect area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. web-dart2js labels Aug 16, 2013
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. web-dart2js
Projects
None yet
Development

No branches or pull requests

5 participants