-
Notifications
You must be signed in to change notification settings - Fork 25.1k
Add Array.push API
#53762
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Add Array.push API
#53762
Conversation
|
@tsaichien has exported this pull request. If you are a Meta employee, you can view the originating diff in D82254334. |
7740e7e to
4d88852
Compare
Summary: Currently, the size of a jsi `Array` is immutable. Once the `Array` is created, users can only set the element at an existing index, but not append to it. This change adds an `Array::push` API to improve ergonomics. Changelog: [Internal] Differential Revision: D82254334
|
@tsaichien has exported this pull request. If you are a Meta employee, you can view the originating diff in D82254334. |
4d88852 to
cc37350
Compare
Summary: Currently, the size of a jsi `Array` is immutable. Once the `Array` is created, users can only set the element at an existing index, but not append to it. This change adds an `Array::push` API to improve ergonomics. Changelog: [Internal] Differential Revision: D82254334
|
@tsaichien has exported this pull request. If you are a Meta employee, you can view the originating diff in D82254334. |
Summary: Currently, the size of a jsi `Array` is immutable. Once the `Array` is created, users can only set the element at an existing index, but not append to it. This change adds an `Array::push` API to improve ergonomics. Changelog: [Internal] Differential Revision: D82254334
cc37350 to
d05a581
Compare
|
@tsaichien has exported this pull request. If you are a Meta employee, you can view the originating diff in D82254334. |
d05a581 to
ed3014b
Compare
Summary: Currently, the size of a jsi `Array` is immutable. Once the `Array` is created, users can only set the element at an existing index, but not append to it. This change adds an `Array::push` API to improve ergonomics. Changelog: [Internal] Differential Revision: D82254334
|
@tsaichien has exported this pull request. If you are a Meta employee, you can view the originating diff in D82254334. |
Summary: Currently, the size of a jsi `Array` is immutable. Once the `Array` is created, users can only set the element at an existing index, but not append to it. This change adds an `Array::push` API to improve ergonomics. Changelog: [Internal] Reviewed By: lavenzg Differential Revision: D82254334
ed3014b to
a044b4f
Compare
|
@tsaichien has exported this pull request. If you are a Meta employee, you can view the originating Diff in D82254334. |
a044b4f to
484991d
Compare
Summary: Currently, the size of a jsi `Array` is immutable. Once the `Array` is created, users can only set the element at an existing index, but not append to it. This change adds an `Array::push` API to improve ergonomics. Changelog: [Internal] Reviewed By: lavenzg Differential Revision: D82254334
484991d to
688b5a0
Compare
Summary: Currently, the size of a jsi `Array` is immutable. Once the `Array` is created, users can only set the element at an existing index, but not append to it. This change adds an `Array::push` API to improve ergonomics. Changelog: [Internal] Reviewed By: lavenzg Differential Revision: D82254334
688b5a0 to
31e97b6
Compare
Summary: Currently, the size of a jsi `Array` is immutable. Once the `Array` is created, users can only set the element at an existing index, but not append to it. This change adds an `Array::push` API to improve ergonomics. Changelog: [Internal] Differential Revision: D82254334
Summary: Currently, the size of a jsi `Array` is immutable. Once the `Array` is created, users can only set the element at an existing index, but not append to it. This change adds an `Array::push` API to improve ergonomics. Changelog: [Internal] Differential Revision: D82254334
31e97b6 to
a6f394f
Compare
Summary: After reviewing the feasibility of adding new Runtime functionality, we realized the current state of `Runtime` makes it hard/inconvenient to add new basic functionality. It requires creating a new interface (e.g. `IRuntime2`) to declare the new APIs. Then, everywhere it is needed, both the original `Runtime` and `IRuntime2` needs to passed in to access both APIs. After discussion, we decided to introduce `IRuntime`, which will declare most of the `Runtime` functionalities. All "protected" APIs, which made sense before interfaces were introduced, will now be public. The protected static methods of `Runtime` will remain as is for the Friends to access. After we decided to cut "stable" (after incoming JSI improvements), new functionalities will be added in `IRuntime2` interface. `IRuntime2` will inherit `IRuntime` to make existing functionality easy to access. Differential Revision: D89093651
Summary: Currently, the size of a jsi `Array` is immutable. Once the `Array` is created, users can only set the element at an existing index, but not append to it. This change adds an `Array::push` API to improve ergonomics. Changelog: [Internal] Differential Revision: D82254334
a6f394f to
f7224f3
Compare
Summary:
Currently, the size of a jsi
Arrayis immutable. Once theArrayiscreated, users can only set the element at an existing index, but not
append to it.
This change adds an
Array::pushAPI to improve ergonomics.Changelog: [Internal]
Differential Revision: D82254334