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

Issue 2983433002: Remove unused native methods ByteData_ToEndian<T> (Closed)

Created:
3 years, 5 months ago by alexmarkov
Modified:
3 years, 5 months ago
CC:
reviews_dartlang.org, vm-dev_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Remove unused native methods ByteData_ToEndian<T> It seems like these native methods are defined in C++ code but not used from Dart code. This CL cleans them up. R=vegorov@google.com, zra@google.com Digging into history reveals that the following CL replaced these native methods with Dart implementation: https://codereview.chromium.org//860963002 Committed: https://github.com/dart-lang/sdk/commit/031533f04d9ad2578a9175b0de7a49b026d61a3c

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -124 lines) Patch
M runtime/lib/typed_data.cc View 1 chunk +0 lines, -116 lines 0 comments Download
M runtime/vm/bootstrap_natives.h View 1 chunk +0 lines, -8 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
alexmarkov
3 years, 5 months ago (2017-07-13 15:14:07 UTC) #2
Vyacheslav Egorov (Google)
lgtm
3 years, 5 months ago (2017-07-13 15:16:03 UTC) #3
zra
lgtm
3 years, 5 months ago (2017-07-13 15:54:17 UTC) #4
alexmarkov
3 years, 5 months ago (2017-07-13 17:08:51 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
031533f04d9ad2578a9175b0de7a49b026d61a3c (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698