sdk icon indicating copy to clipboard operation
sdk copied to clipboard

`_IndexAssembler.assemble`: optimize using `toList(growable: false)`

Open gmpassos opened this issue 1 year ago • 1 comments

The assemble method is a hotspot for List generation, and a fixed-length list is faster to generate.


  • [✅] I’ve reviewed the contributor guide and applied the relevant portions to this PR.

gmpassos avatar Aug 21 '24 18:08 gmpassos

Thank you for your contribution! This project uses Gerrit for code reviews. Your pull request has automatically been converted into a code review at:

https://dart-review.googlesource.com/c/sdk/+/381780

Please wait for a developer to review your code review at the above link; you can speed up the review if you sign into Gerrit and manually add a reviewer that has recently worked on the relevant code. See CONTRIBUTING.md to learn how to upload changes to Gerrit directly.

Additional commits pushed to this PR will update both the PR and the corresponding Gerrit CL. After the review is complete on the CL, your reviewer will merge the CL (automatically closing this PR).

copybara-service[bot] avatar Aug 21 '24 18:08 copybara-service[bot]

@gmpassos @bwilkerson what is the status of this change?

mraleph avatar Sep 11 '24 12:09 mraleph

The last update on https://dart-review.googlesource.com/c/sdk/+/381780 is:

This CL has passed the run

gmpassos avatar Sep 11 '24 20:09 gmpassos

https://dart-review.googlesource.com/c/sdk/+/381780 has been updated with the latest commits from this pull request.

copybara-service[bot] avatar Sep 17 '24 21:09 copybara-service[bot]

https://dart-review.googlesource.com/c/sdk/+/381780 has been updated with the latest commits from this pull request.

copybara-service[bot] avatar Sep 17 '24 21:09 copybara-service[bot]

@bwilkerson @scheglov same here, please make a call and either accept (+1 & +2 the CL) or reject the PR. Thanks!

mraleph avatar Sep 23 '24 12:09 mraleph