Remove unnecessary casts.

Remove unnecessary casts.

R=dmitryas@google.com

Committed: https://github.com/dart-lang/sdk/commit/0d83d616015212ef652f4cc26c7cc4f58565fa97

[ahe, 2017-03-16 09:37:08]

ahe@google.com changed reviewers:
+ dmitryas@google.com

[ahe, 2017-03-16 09:37:09]

https://codereview.chromium.org/2752823003/diff/1/pkg/kernel/lib/transformati...
File pkg/kernel/lib/transformations/closure/converter.dart (right):

https://codereview.chromium.org/2752823003/diff/1/pkg/kernel/lib/transformati...
pkg/kernel/lib/transformations/closure/converter.dart:679: if (node is Class &&
node.name == closureClassName) {
Because of the test "node is Class", the compiler now knows that node has the
type Class. As long as you don't assign to node in the code that's guarded by
this test (or in a closure anywhere), there's no need for a cast.

[Dmitry Stefantsov, 2017-03-16 09:42:10]

LGTM

https://codereview.chromium.org/2752823003/diff/1/pkg/kernel/lib/transformati...
File pkg/kernel/lib/transformations/closure/converter.dart (right):

https://codereview.chromium.org/2752823003/diff/1/pkg/kernel/lib/transformati...
pkg/kernel/lib/transformations/closure/converter.dart:679: if (node is Class &&
node.name == closureClassName) {
On 2017/03/16 09:37:09, ahe wrote:
> Because of the test "node is Class", the compiler now knows that node has the
> type Class. As long as you don't assign to node in the code that's guarded by
> this test (or in a closure anywhere), there's no need for a cast.

Thanks!

[ahe, 2017-03-16 09:49:53]

Description was changed from

==========
Remove unnecessary casts.
==========

to

==========
Remove unnecessary casts.

R=dmitryas@google.com

Committed:
https://github.com/dart-lang/sdk/commit/0d83d616015212ef652f4cc26c7cc4f58565fa97
==========

[ahe, 2017-03-16 09:49:55]

Committed patchset #1 (id:1) manually as
0d83d616015212ef652f4cc26c7cc4f58565fa97 (presubmit successful).