Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(270)

Issue 2200263004: Disallow assigning null to non-nullable variables (Closed)

Created:
4 years, 4 months ago by stanm
Modified:
4 years, 4 months ago
Reviewers:
Jennifer Messerly
CC:
reviews_dartlang.org
Base URL:
https://github.com/dart-lang/sdk@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Disallow assigning null to non-nullable variables BUG= R=jmesserly@google.com Committed: https://github.com/dart-lang/sdk/commit/3237d789d1a6bac1c0f6bd9cabcb9e0f5124637a

Patch Set 1 #

Patch Set 2 : Add more tests #

Patch Set 3 : Move _getTypeFullyQualifiedName to type_system.dart #

Patch Set 4 : Remove obsolete comment #

Total comments: 4

Patch Set 5 : Address lgtm comment #

Messages

Total messages: 6 (2 generated)
stanm
4 years, 4 months ago (2016-08-02 23:42:32 UTC) #2
Jennifer Messerly
lgtm https://codereview.chromium.org/2200263004/diff/60001/pkg/analyzer/lib/src/generated/type_system.dart File pkg/analyzer/lib/src/generated/type_system.dart (right): https://codereview.chromium.org/2200263004/diff/60001/pkg/analyzer/lib/src/generated/type_system.dart#newcode773 pkg/analyzer/lib/src/generated/type_system.dart:773: return "${type?.element?.library?.identifier},$type"; .source.uri ? https://codereview.chromium.org/2200263004/diff/60001/pkg/analyzer/test/src/task/strong/non_null_checker_test.dart File pkg/analyzer/test/src/task/strong/non_null_checker_test.dart (right): ...
4 years, 4 months ago (2016-08-03 18:06:56 UTC) #3
stanm
https://codereview.chromium.org/2200263004/diff/60001/pkg/analyzer/lib/src/generated/type_system.dart File pkg/analyzer/lib/src/generated/type_system.dart (right): https://codereview.chromium.org/2200263004/diff/60001/pkg/analyzer/lib/src/generated/type_system.dart#newcode773 pkg/analyzer/lib/src/generated/type_system.dart:773: return "${type?.element?.library?.identifier},$type"; On 2016/08/03 18:06:56, John Messerly wrote: > ...
4 years, 4 months ago (2016-08-03 18:46:21 UTC) #4
stanm
4 years, 4 months ago (2016-08-03 18:47:18 UTC) #6
Message was sent while issue was closed.
Committed patchset #5 (id:80001) manually as
3237d789d1a6bac1c0f6bd9cabcb9e0f5124637a (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698