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

Issue 3005533002: Fix the return type of String.split in DDC to be correct (Closed)

Created:
3 years, 4 months ago by Jennifer Messerly
Modified:
3 years, 3 months ago
Reviewers:
Leaf, vsm
CC:
reviews_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Fix the return type of String.split in DDC to be correct It wasn't actually a List<String>, so it could fail a strong mode cast, and also permitted non-Strings to be added. R=leafp@google.com Committed: https://github.com/dart-lang/sdk/commit/d0072fa73fcb5421e49ba7e86818b891c70303d8

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+21 lines, -5 lines) Patch
M pkg/dev_compiler/tool/input_sdk/private/js_string.dart View 1 chunk +2 lines, -2 lines 0 comments Download
M tests/corelib_2/corelib_2.status View 2 chunks +4 lines, -0 lines 1 comment Download
M tests/corelib_2/string_split_test.dart View 2 chunks +15 lines, -3 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
Jennifer Messerly
3 years, 4 months ago (2017-08-24 17:10:08 UTC) #2
Leaf
lgtm https://codereview.chromium.org/3005533002/diff/1/tests/corelib_2/corelib_2.status File tests/corelib_2/corelib_2.status (right): https://codereview.chromium.org/3005533002/diff/1/tests/corelib_2/corelib_2.status#newcode183 tests/corelib_2/corelib_2.status:183: int_parse_radix_test/02: RuntimeError File a bug for these?
3 years, 4 months ago (2017-08-24 17:30:09 UTC) #3
Jennifer Messerly
Committed patchset #1 (id:1) manually as d0072fa73fcb5421e49ba7e86818b891c70303d8 (presubmit successful).
3 years, 3 months ago (2017-08-25 18:25:02 UTC) #5
Jennifer Messerly
3 years, 3 months ago (2017-08-25 18:27:30 UTC) #6
Message was sent while issue was closed.
thanks! i filed https://github.com/dart-lang/sdk/issues/30548

Powered by Google App Engine
This is Rietveld 408576698