| Index: src/elements.h
|
| ===================================================================
|
| --- src/elements.h (revision 11528)
|
| +++ src/elements.h (working copy)
|
| @@ -60,18 +60,19 @@
|
| // can optionally pass in the backing store to use for the check, which must
|
| // be compatible with the ElementsKind of the ElementsAccessor. If
|
| // backing_store is NULL, the holder->elements() is used as the backing store.
|
| - virtual MaybeObject* Get(Object* receiver,
|
| - JSObject* holder,
|
| - uint32_t key,
|
| - FixedArrayBase* backing_store = NULL) = 0;
|
| + MUST_USE_RESULT virtual MaybeObject* Get(
|
| + Object* receiver,
|
| + JSObject* holder,
|
| + uint32_t key,
|
| + FixedArrayBase* backing_store = NULL) = 0;
|
|
|
| // Modifies the length data property as specified for JSArrays and resizes the
|
| // underlying backing store accordingly. The method honors the semantics of
|
| // changing array sizes as defined in EcmaScript 5.1 15.4.5.2, i.e. array that
|
| // have non-deletable elements can only be shrunk to the size of highest
|
| // element that is non-deletable.
|
| - virtual MaybeObject* SetLength(JSArray* holder,
|
| - Object* new_length) = 0;
|
| + MUST_USE_RESULT virtual MaybeObject* SetLength(JSArray* holder,
|
| + Object* new_length) = 0;
|
|
|
| // Modifies both the length and capacity of a JSArray, resizing the underlying
|
| // backing store as necessary. This method does NOT honor the semantics of
|
| @@ -79,14 +80,14 @@
|
| // elements. This method should only be called for array expansion OR by
|
| // runtime JavaScript code that use InternalArrays and don't care about
|
| // EcmaScript 5.1 semantics.
|
| - virtual MaybeObject* SetCapacityAndLength(JSArray* array,
|
| - int capacity,
|
| - int length) = 0;
|
| + MUST_USE_RESULT virtual MaybeObject* SetCapacityAndLength(JSArray* array,
|
| + int capacity,
|
| + int length) = 0;
|
|
|
| // Deletes an element in an object, returning a new elements backing store.
|
| - virtual MaybeObject* Delete(JSObject* holder,
|
| - uint32_t key,
|
| - JSReceiver::DeleteMode mode) = 0;
|
| + MUST_USE_RESULT virtual MaybeObject* Delete(JSObject* holder,
|
| + uint32_t key,
|
| + JSReceiver::DeleteMode mode) = 0;
|
|
|
| // If kCopyToEnd is specified as the copy_size to CopyElements, it copies all
|
| // of elements from source after source_start to the destination array.
|
| @@ -101,26 +102,28 @@
|
| // the source JSObject or JSArray in source_holder. If the holder's backing
|
| // store is available, it can be passed in source and source_holder is
|
| // ignored.
|
| - virtual MaybeObject* CopyElements(JSObject* source_holder,
|
| - uint32_t source_start,
|
| - FixedArrayBase* destination,
|
| - ElementsKind destination_kind,
|
| - uint32_t destination_start,
|
| - int copy_size,
|
| - FixedArrayBase* source = NULL) = 0;
|
| + MUST_USE_RESULT virtual MaybeObject* CopyElements(
|
| + JSObject* source_holder,
|
| + uint32_t source_start,
|
| + FixedArrayBase* destination,
|
| + ElementsKind destination_kind,
|
| + uint32_t destination_start,
|
| + int copy_size,
|
| + FixedArrayBase* source = NULL) = 0;
|
|
|
| - MaybeObject* CopyElements(JSObject* from_holder,
|
| - FixedArrayBase* to,
|
| - ElementsKind to_kind,
|
| - FixedArrayBase* from = NULL) {
|
| + MUST_USE_RESULT MaybeObject* CopyElements(JSObject* from_holder,
|
| + FixedArrayBase* to,
|
| + ElementsKind to_kind,
|
| + FixedArrayBase* from = NULL) {
|
| return CopyElements(from_holder, 0, to, to_kind, 0,
|
| kCopyToEndAndInitializeToHole, from);
|
| }
|
|
|
| - virtual MaybeObject* AddElementsToFixedArray(Object* receiver,
|
| - JSObject* holder,
|
| - FixedArray* to,
|
| - FixedArrayBase* from = NULL) = 0;
|
| + MUST_USE_RESULT virtual MaybeObject* AddElementsToFixedArray(
|
| + Object* receiver,
|
| + JSObject* holder,
|
| + FixedArray* to,
|
| + FixedArrayBase* from = NULL) = 0;
|
|
|
| // Returns a shared ElementsAccessor for the specified ElementsKind.
|
| static ElementsAccessor* ForKind(ElementsKind elements_kind) {
|
|
|