Skip to content

Remove unnecessary ToArray call #44864

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Nov 3, 2022
Merged

Remove unnecessary ToArray call #44864

merged 1 commit into from
Nov 3, 2022

Conversation

BrennanConroy
Copy link
Member

Change is almost not worth making 😆
Saves 0 or 2 bytes in the common case. But since it's a super simple small change I decided to do it...

@BrennanConroy
Remove unnecessary ToArray call
624f764 
Change is almost not worth making 😆 
Saves 0 or 2 bytes in the common case. But since it's a super simple small change I decided to do it...
@ghost ghost added the area-runtime label Nov 3, 2022
@BrennanConroy BrennanConroy merged commit 2e29720 into main Nov 3, 2022
@BrennanConroy BrennanConroy deleted the brecon/toarray branch November 3, 2022 20:33
@ghost ghost added this to the 8.0-preview1 milestone Nov 3, 2022
@amcasey amcasey added area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions and removed area-runtime labels Aug 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Tratcher Tratcher Tratcher approved these changes

@halter73 halter73 Awaiting requested review from halter73 halter73 is a code owner

Assignees
No one assigned
Labels
area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions Perf
Projects
None yet
Milestone
8.0-preview1
Development

Successfully merging this pull request may close these issues.
3 participants
@BrennanConroy @Tratcher @amcasey