-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Platform.isLinux and company should be const #35705
Comments
Hi. I'm the new maintainer of dart:io. I actually ran into this issue the other day, I agree it would be nice for them to be const. |
Seems to be related to #31969 |
Any idea on where this issue is in the priorities for the SDK? This would be pretty useful, especially now that flutter is expanding to more platforms. |
Hi Greg. I'm hoping to do some work on this issue this quarter so flutter for web is better supported. I'm writing up a plan and will be sharing it. |
@sortie Please re-evaluate this and do one of the following:
|
After 2c0484c and a4cb6ab, this should be the case for AOT-compiled programs in Dart. For Flutter, after flutter/flutter#127567 landed. |
We'd like to have code that does something like this:
We only want to ship the maps that apply to the current platform; we'd like the others to tree-shake out.
The text was updated successfully, but these errors were encountered: