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

Issue 9769013: Narrow when storing to a byte array and implement uint64 get and set. (Closed)

Created:
8 years, 9 months ago by cshapiro
Modified:
8 years, 9 months ago
Reviewers:
jjb, siva
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Narrow when storing to a byte array and implement uint64 get and set. Previously, the byte array get and set routines for integers would not correctly narrow an integer when the width of a value being stored was greater than the element width. This change implements the behavior in the Web IDL specification. In addition, an implementation for uint64 get and set is now provided. BUG=2042 Committed: https://code.google.com/p/dart/source/detail?r=5738

Patch Set 1 #

Total comments: 2

Patch Set 2 : address review comments #

Unified diffs Side-by-side diffs Delta from patch set Stats (+100 lines, -27 lines) Patch
M runtime/lib/byte_array.cc View 1 4 chunks +47 lines, -10 lines 0 comments Download
M runtime/lib/byte_array.dart View 1 17 chunks +53 lines, -17 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
cshapiro
Should I add a TODO regarding the shift-and-subtract operations? I cannot do the more sensible ...
8 years, 9 months ago (2012-03-21 05:52:17 UTC) #1
siva
Sure please add the TODO so that we remember to change to the new form ...
8 years, 9 months ago (2012-03-22 00:27:20 UTC) #2
cshapiro
PTAL I've added comments after each use of the shift-and-subtract. It is ugly but that ...
8 years, 9 months ago (2012-03-22 00:47:46 UTC) #3
siva
lgtm
8 years, 9 months ago (2012-03-22 01:07:59 UTC) #4
siva
8 years, 9 months ago (2012-03-22 01:08:05 UTC) #5
lgtm

Powered by Google App Engine
This is Rietveld 408576698