-
Notifications
You must be signed in to change notification settings - Fork 209
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
Specify null safety #2605
Open
eernstg
wants to merge
89
commits into
main
Choose a base branch
from
specify_null_safety_sep21
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Specify null safety #2605
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Visit the preview URL for this PR (updated for commit 37acce6): https://dart-specification--pr2605-specify-null-safety-50hej3os.web.app (expires Mon, 18 Sep 2023 13:58:21 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 6941ecd630c4f067ff3d02708a45ae0f0a42b88a |
581e7c5
to
8f1fd57
Compare
8f1fd57
to
010c13b
Compare
43ec6a7
to
73ce64b
Compare
73ce64b
to
1d740a9
Compare
9b4f226
to
7bec6da
Compare
179dee5
to
7e6cd3d
Compare
7e6cd3d
to
0ccb1e8
Compare
1c7d5ac
to
61717df
Compare
57134d6
to
a07718a
Compare
72872b5
to
8f6a5e0
Compare
Woohoo! Part of this PR has now been landed: #2052. |
8f6a5e0
to
df42dfe
Compare
eernstg
referenced
this pull request
Jul 21, 2023
…tions' (#2052) Integrate the null-safety aware sections about variables and local variable declarations in the the language specification. Also updates the terminology to talk about 'static X' meaning a member declaration that includes the keyword `static` (or a member whose declaration includes `static`). This means that we avoid having the extremely confusing phrase 'static variable' (meaning a top-level variable or a variable whose declaration includes `static`). The new meaning of 'static variable' is simply a variable whose declaration includes `static`. Eliminated the single occurrence of 'static function' (now using 'static member', which is well defined). Similarly, updates the terminology to avoid the words 'mutable' and 'immutable' (about variables). The reason for this is that the concept of being immutable is confusing now that we have `late final v;`: That declaration induces a setter, but it is semantically immutable (because it may be initialized, but it will never update the initial value to a different value).
c7a111a
to
e922a06
Compare
04f0430
to
37acce6
Compare
37acce6
to
7aea1e0
Compare
b68a613
to
8346903
Compare
…new/const C<T>()` is still an error when `C<T>` is super-bounded
…nal and named parameters, not just parameters that are both
f3077a4
to
f6d64be
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is the full PR that integrates the null-safety feature into the language specification. It is not intended to be landed as such: Parts of it are currently being reviewed, and further parts will be reviewed later on, such that all of this gets landed, but avoiding a review which is too large to get done in one fell swoop.
Parts under review (as of Nov 2 2022): #2052, #2023.
This PR has been uploaded at this point in order to make it more convenient to refer to specific locations in the proposed new text. When each of the parts lands it is expected that this PR is rebased, and piece by piece of it will disappear because some of the changes in this PR have already been made by those partial PRs.