flipper icon indicating copy to clipboard operation
flipper copied to clipboard

Bump react-virtualized-auto-sizer and @types/react-virtualized-auto-sizer in /desktop

Open dependabot[bot] opened this issue 10 months ago • 8 comments

Summary

This PR implements ES2023 TypedArray.prototype.toReversed method. This is a follow-up of PR https://github.com/facebook/hermes/pull/1286 which added with method on standard Array prototype.

See also:

  • PR for TypedArray.prototype.with: #1365
  • PR for TypedArray.prototype.toSorted: #1367

I am creating separate PRs for each method in case there are any suggestions or change requests from your side, so that PRs are more independent.

Test Plan

Code is annotated with algorithm steps from EcmaScript specification for easier verification and maintenance. I also added tests to verify that methods work as intended. There might be some more edge cases that might be covered based on your experience.

$ echo "new Int8Array([0, 1, 2, 3]).toReversed()" | ./bin/hermes
# >> Int8Array [ 3, 2, 1, 0 ]

dependabot[bot] avatar Apr 08 '24 13:04 dependabot[bot]

@dannysu has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

facebook-github-bot avatar Mar 29 '24 23:03 facebook-github-bot

@robik has updated the pull request. You must reimport the pull request before landing.

facebook-github-bot avatar May 23 '24 10:05 facebook-github-bot

@dannysu has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

facebook-github-bot avatar Jun 13 '24 20:06 facebook-github-bot

@robik has updated the pull request. You must reimport the pull request before landing.

facebook-github-bot avatar Jun 18 '24 11:06 facebook-github-bot

For the other PRs, we should eliminate setOwnIndexed or setToCopyOfTypedArray as well, since we know exactly the the underlying data array and the source/target array buffer have the same type.

lavenzg avatar Jun 18 '24 20:06 lavenzg

Sure thing! I just wanted to make sure that the current approach is correct first. If it is, I can apply it to other methods :)

robik avatar Jun 19 '24 08:06 robik

@robik has updated the pull request. You must reimport the pull request before landing.

facebook-github-bot avatar Jun 28 '24 08:06 facebook-github-bot

@robik has updated the pull request. You must reimport the pull request before landing.

facebook-github-bot avatar Jun 28 '24 14:06 facebook-github-bot