Use offsetInBytes and not index in the get/set accessors of views.

runtime/lib/typeddata.dart

 // Copyright (c) 2013, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 // patch classes for Int8List ..... Float64List and ByteData implementations.
 
 patch class Int8List {
   /* patch */ factory Int8List(int length) {
     return new _Int8Array(length);
   }
@@ skipping 457 matching lines @@
   }
 
 
   // Methods implementing the collection interface.
 
   int get length native "TypedData_length";
 
 
   // Internal utility methods.
 
-  int _getInt8(int index) native "TypedData_GetInt8";
-  void _setInt8(int index, int value) native "TypedData_SetInt8";
+  int _getInt8(int offsetInBytes) native "TypedData_GetInt8";
+  void _setInt8(int offsetInBytes, int value) native "TypedData_SetInt8";
 
-  int _getUint8(int index) native "TypedData_GetUint8";
-  void _setUint8(int index, int value) native "TypedData_SetUint8";
+  int _getUint8(int offsetInBytes) native "TypedData_GetUint8";
+  void _setUint8(int offsetInBytes, int value) native "TypedData_SetUint8";
 
-  int _getInt16(int index) native "TypedData_GetInt16";
-  void _setInt16(int index, int value) native "TypedData_SetInt16";
+  int _getIndexedInt16(int index) {

[Florian Schneider, 2013/03/14 10:12:00]

It does not make a huge difference, but it may not be necessary to have
_getIndexedInt16 in the base class, but only in Int16Array. Same for the other
_getIndexedX/_setIndexedX functions.

[siva, 2013/03/14 11:16:34]

True, I will move them to the respective classes.

On 2013/03/14 10:12:00, Florian Schneider wrote:

+    return _getInt16(index * Int16List.BYTES_PER_ELEMENT);
+  }
+  int _getInt16(int offsetInBytes) native "TypedData_GetInt16";
+  void _setIndexedInt16(int index, int value) {
+    _setInt16(index * Int16List.BYTES_PER_ELEMENT, value);
+  }
+  void _setInt16(int offsetInBytes, int value) native "TypedData_SetInt16";
 
-  int _getUint16(int index) native "TypedData_GetUint16";
-  void _setUint16(int index, int value) native "TypedData_SetUint16";
+  int _getIndexedUint16(int index) {
+    return _getUint16(index * Uint16List.BYTES_PER_ELEMENT);
+  }
+  int _getUint16(int offsetInBytes) native "TypedData_GetUint16";
+  void _setIndexedUint16(int index, int value) {
+    _setUint16(index * Uint16List.BYTES_PER_ELEMENT, value);
+  }
+  void _setUint16(int offsetInBytes, int value) native "TypedData_SetUint16";
 
-  int _getInt32(int index) native "TypedData_GetInt32";
-  void _setInt32(int index, int value) native "TypedData_SetInt32";
+  int _getIndexedInt32(int index) {
+    return _getInt32(index * Int32List.BYTES_PER_ELEMENT);
+  }
+  int _getInt32(int offsetInBytes) native "TypedData_GetInt32";
+  void _setIndexedInt32(int index, int value) {
+    _setInt32(index * Int32List.BYTES_PER_ELEMENT, value);
+  }
+  void _setInt32(int offsetInBytes, int value) native "TypedData_SetInt32";
 
-  int _getUint32(int index) native "TypedData_GetUint32";
-  void _setUint32(int index, int value) native "TypedData_SetUint32";
+  int _getIndexedUint32(int index) {
+    return _getUint32(index * Uint32List.BYTES_PER_ELEMENT);
+  }
+  int _getUint32(int offsetInBytes) native "TypedData_GetUint32";
+  void _setIndexedUint32(int index, int value) {
+    _setInt32(index * Uint32List.BYTES_PER_ELEMENT, value);
+  }
+  void _setUint32(int offsetInBytes, int value) native "TypedData_SetUint32";
 
-  int _getInt64(int index) native "TypedData_GetInt64";
-  void _setInt64(int index, int value) native "TypedData_SetInt64";
+  int _getIndexedInt64(int index) {
+    return _getInt64(index * Int64List.BYTES_PER_ELEMENT);
+  }
+  int _getInt64(int offsetInBytes) native "TypedData_GetInt64";
+  void _setIndexedInt64(int index, int value) {
+    _setInt64(index * Int64List.BYTES_PER_ELEMENT, value);
+  }
+  void _setInt64(int offsetInBytes, int value) native "TypedData_SetInt64";
 
-  int _getUint64(int index) native "TypedData_GetUint64";
-  void _setUint64(int index, int value) native "TypedData_SetUint64";
+  int _getIndexedUint64(int index) {
+    return _getUint64(index * Uint64List.BYTES_PER_ELEMENT);
+  }
+  int _getUint64(int offsetInBytes) native "TypedData_GetUint64";
+  void _setIndexedUint64(int index, int value) {
+    _setUint64(index * Uint64List.BYTES_PER_ELEMENT, value);
+  }
+  void _setUint64(int offsetInBytes, int value) native "TypedData_SetUint64";
 
-  double _getFloat32(int index) native "TypedData_GetFloat32";
-  void _setFloat32(int index, double value) native "TypedData_SetFloat32";
+  double _getIndexedFloat32(int index) {
+    return _getFloat32(index * Float32List.BYTES_PER_ELEMENT);
+  }
+  double _getFloat32(int offsetInBytes) native "TypedData_GetFloat32";
+  void _setIndexedFloat32(int index, double value) {
+    _setFloat32(index * Float32List.BYTES_PER_ELEMENT, value);
+  }
+  void _setFloat32(int offsetInBytes, double value)
+      native "TypedData_SetFloat32";
 
-  double _getFloat64(int index) native "TypedData_GetFloat64";
-  void _setFloat64(int index, double value) native "TypedData_SetFloat64";
+  double _getIndexedFloat64(int index) {
+    return _getFloat64(index * Float64List.BYTES_PER_ELEMENT);
+  }
+  double _getFloat64(int offsetInBytes) native "TypedData_GetFloat64";
+  void _setIndexedFloat64(int index, double value) {
+    _setFloat64(index * Float64List.BYTES_PER_ELEMENT, value);
+  }
+  void _setFloat64(int offsetInBytes, double value)
+      native "TypedData_SetFloat64";
 }
 
 
 class _Int8Array extends _TypedList implements Int8List {
   // Factory constructors.
 
   factory _Int8Array(int length) {
     if (length < 0) {
       String message = "$length must be greater than 0";
       throw new ArgumentError(message);
@@ skipping 189 matching lines @@
   }
 
 
   // Method(s) implementing List interface.
 
   int operator[](int index) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    return _getInt16(index);
+    return _getIndexedInt16(index);
   }
 
   void operator[]=(int index, int value) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    _setInt16(index, _toInt16(value));
+    _setIndexedInt16(index, _toInt16(value));
   }
 
   Iterator<int> get iterator {
     return new _TypedListIterator<int>(this);
   }
 
 
   // Method(s) implementing TypedData interface.
 
   int get elementSizeInBytes {
@@ skipping 32 matching lines @@
   }
 
 
   // Method(s) implementing the List interface.
 
   int operator[](int index) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    return _getUint16(index);
+    return _getIndexedUint16(index);
   }
 
   void operator[]=(int index, int value) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    _setUint16(index, _toUint16(value));
+    _setIndexedUint16(index, _toUint16(value));
   }
 
   Iterator<int> get iterator {
     return new _TypedListIterator<int>(this);
   }
 
 
   // Method(s) implementing the TypedData interface.
 
   int get elementSizeInBytes {
@@ skipping 32 matching lines @@
   }
 
 
   // Method(s) implementing the List interface.
 
   int operator[](int index) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    return _getInt32(index);
+    return _getIndexedInt32(index);
   }
 
   void operator[]=(int index, int value) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    _setInt32(index, _toInt32(value));
+    _setIndexedInt32(index, _toInt32(value));
   }
 
   Iterator<int> get iterator {
     return new _TypedListIterator<int>(this);
   }
 
 
   // Method(s) implementing TypedData interface.
 
   int get elementSizeInBytes {
@@ skipping 32 matching lines @@
   }
 
 
   // Method(s) implementing the List interface.
 
   int operator[](int index) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    return _getUint32(index);
+    return _getIndexedUint32(index);
   }
 
   void operator[]=(int index, int value) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    _setUint32(index, _toUint32(value));
+    _setIndexedUint32(index, _toUint32(value));
   }
 
   Iterator<int> get iterator {
     return new _TypedListIterator<int>(this);
   }
 
 
   // Method(s) implementing the TypedData interface.
 
   int get elementSizeInBytes {
@@ skipping 32 matching lines @@
   }
 
 
   // Method(s) implementing the List interface.
 
   int operator[](int index) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    return _getInt64(index);
+    return _getIndexedInt64(index);
   }
 
   void operator[]=(int index, int value) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    _setInt64(index, _toInt64(value));
+    _setIndexedInt64(index, _toInt64(value));
   }
 
   Iterator<int> get iterator {
     return new _TypedListIterator<int>(this);
   }
 
 
   // Method(s) implementing the TypedData interface.
 
   int get elementSizeInBytes {
@@ skipping 32 matching lines @@
   }
 
 
   // Method(s) implementing the List interface.
 
   int operator[](int index) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    return _getUint64(index);
+    return _getIndexedUint64(index);
   }
 
   void operator[]=(int index, int value) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    _setUint64(index, _toUint64(value));
+    _setIndexedUint64(index, _toUint64(value));
   }
 
   Iterator<int> get iterator {
     return new _TypedListIterator<int>(this);
   }
 
 
   // Method(s) implementing the TypedData interface.
 
   int get elementSizeInBytes {
@@ skipping 32 matching lines @@
   }
 
 
   // Method(s) implementing the List interface.
 
   double operator[](int index) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    return _getFloat32(index);
+    return _getIndexedFloat32(index);
   }
 
   void operator[]=(int index, double value) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    _setFloat32(index, value);
+    _setIndexedFloat32(index, value);
   }
 
   Iterator<double> get iterator {
     return new _TypedListIterator<double>(this);
   }
 
 
   // Method(s) implementing the TypedData interface.
 
   int get elementSizeInBytes {
@@ skipping 32 matching lines @@
   }
 
 
   // Method(s) implementing the List interface.
 
   double operator[](int index) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    return _getFloat64(index);
+    return _getIndexedFloat64(index);
   }
 
   void operator[]=(int index, double value) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    _setFloat64(index, value);
+    _setIndexedFloat64(index, value);
   }
 
   Iterator<double> get iterator {
     return new _TypedListIterator<double>(this);
   }
 
 
   // Method(s) implementing the TypedData interface.
 
   int get elementSizeInBytes {
@@ skipping 181 matching lines @@
   }
 
 
   // Method(s) implementing the List interface.
 
   int operator[](int index) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    return _getInt16(index);
+    return _getIndexedInt16(index);
   }
 
   void operator[]=(int index, int value) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    _setInt16(index, _toInt16(value));
+    _setIndexedInt16(index, _toInt16(value));
   }
 
   Iterator<int> get iterator {
     return new _TypedListIterator<int>(this);
   }
 
 
   // Method(s) implementing the TypedData interface.
 
   int get elementSizeInBytes {
@@ skipping 24 matching lines @@
   }
 
 
   // Method(s) implementing the List interface.
 
   int operator[](int index) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    return _getUint16(index);
+    return _getIndexedUint16(index);
   }
 
   void operator[]=(int index, int value) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    _setUint16(index, _toUint16(value));
+    _setIndexedUint16(index, _toUint16(value));
   }
 
   Iterator<int> get iterator {
     return new _TypedListIterator<int>(this);
   }
 
 
   // Method(s) implementing the TypedData interface.
 
   int get elementSizeInBytes {
@@ skipping 24 matching lines @@
   }
 
 
   // Method(s) implementing the List interface.
 
   int operator[](int index) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    return _getInt32(index);
+    return _getIndexedInt32(index);
   }
 
   void operator[]=(int index, int value) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    _setInt32(index, _toInt32(value));
+    _setIndexedInt32(index, _toInt32(value));
   }
 
   Iterator<int> get iterator {
     return new _TypedListIterator<int>(this);
   }
 
 
   // Method(s) implementing the TypedData interface.
 
   int get elementSizeInBytes {
@@ skipping 24 matching lines @@
   }
 
 
   // Method(s) implementing the List interface.
 
   int operator[](int index) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    return _getUint32(index);
+    return _getIndexedUint32(index);
   }
 
   void operator[]=(int index, int value) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    _setUint32(index, _toUint32(value));
+    _setIndexedUint32(index, _toUint32(value));
   }
 
   Iterator<int> get iterator {
     return new _TypedListIterator<int>(this);
   }
 
 
   // Method(s) implementing the TypedData interface.
 
   int get elementSizeInBytes {
@@ skipping 24 matching lines @@
   }
 
 
   // Method(s) implementing the List interface.
 
   int operator[](int index) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    return _getInt64(index);
+    return _getIndexedInt64(index);
   }
 
   void operator[]=(int index, int value) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    _setInt64(index, _toInt64(value));
+    _setIndexedInt64(index, _toInt64(value));
   }
 
   Iterator<int> get iterator {
     return new _TypedListIterator<int>(this);
   }
 
 
   // Method(s) implementing the TypedData interface.
 
   int get elementSizeInBytes {
@@ skipping 24 matching lines @@
   }
 
 
   // Method(s) implementing the List interface.
 
   int operator[](int index) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    return _getUint64(index);
+    return _getIndexedUint64(index);
   }
 
   void operator[]=(int index, int value) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    _setUint64(index, _toUint64(value));
+    _setIndexedUint64(index, _toUint64(value));
   }
 
   Iterator<int> get iterator {
     return new _TypedListIterator<int>(this);
   }
 
 
   // Method(s) implementing the TypedData interface.
 
   Uint64List _createList(int length) {
@@ skipping 24 matching lines @@
   }
 
 
   // Method(s) implementing the List interface.
 
   double operator[](int index) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    return _getFloat32(index);
+    return _getIndexedFloat32(index);
   }
 
   void operator[]=(int index, double value) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    _setFloat32(index, value);
+    _setIndexedFloat32(index, value);
   }
 
   Iterator<double> get iterator {
     return new _TypedListIterator<double>(this);
   }
 
 
   // Method(s) implementing the TypedData interface.
 
   int get elementSizeInBytes {
@@ skipping 24 matching lines @@
   }
 
 
   // Method(s) implementing the List interface.
 
   double operator[](int index) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    return _getFloat64(index);
+    return _getIndexedFloat64(index);
   }
 
   void operator[]=(int index, double value) {
     if (index < 0 || index >= length) {
       String message = "$index must be in the range [0..$length)";
       throw new RangeError(message);
     }
-    _setFloat64(index, value);
+    _setIndexedFloat64(index, value);
   }
 
   Iterator<double> get iterator {
     return new _TypedListIterator<double>(this);
   }
 
 
   // Method(s) implementing the TypedData interface.
 
   int get elementSizeInBytes {
@@ skipping 738 matching lines @@
   }
 }
 
 
 int _defaultIfNull(object, value) {
   if (object == null) {
     return value;
   }
   return object;
 }