Issue 2859133003: Find native classes with mixins in NativeClassFinder

Issue 2859133003: Find native classes with mixins in NativeClassFinder (Closed)

Created:
3 years, 7 months ago by Johnni Winther

Modified:
3 years, 7 months ago

Reviewers:
Siggi Cherem (dart-lang)

CC:
reviews_dartlang.org

Target Ref:
refs/heads/master

Visibility:
Public.

Find native classes with mixins in NativeClassFinder We were skipping for instance NativeTypedArrayOfDouble because it mixes in ListMixin and FixedLengthListMixin. R=sigmund@google.com Committed: https://github.com/dart-lang/sdk/commit/b7225cf5334b3aeabac4ce07b0a20b4b3beecfe6

Created: 3 years, 7 months ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Stats (+70 lines, -20 lines)			Patch
M	pkg/compiler/lib/src/common_elements.dart	View	1 chunk	+7 lines, -3 lines	0 comments	Download
M	pkg/compiler/lib/src/js_backend/backend.dart	View	1 chunk	+1 line, -3 lines	0 comments	Download
M	pkg/compiler/lib/src/kernel/element_map.dart	View	3 chunks	+23 lines, -10 lines	0 comments	Download
M	pkg/compiler/lib/src/native/enqueue.dart	View	1 chunk	+4 lines, -0 lines	0 comments	Download
M	pkg/compiler/lib/src/native/resolver.dart	View	1 chunk	+3 lines, -1 line	0 comments	Download
M	pkg/compiler/lib/src/resolution/resolution_strategy.dart	View	1 chunk	+9 lines, -2 lines	0 comments	Download
M	tests/compiler/dart2js/kernel/closed_world2_test.dart	View	2 chunks	+22 lines, -0 lines	0 comments	Download
M	tests/compiler/dart2js/kernel/closed_world_test.dart	View	1 chunk	+1 line, -1 line	0 comments	Download

Total messages: 5 (2 generated)

Johnni Winther

3 years, 7 months ago (2017-05-05 08:27:38 UTC) #5

Message was sent while issue was closed.

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