8000 Avoid unnecessary `StringCollection` allocation by xtqqczze · Pull Request #15832 · PowerShell/PowerShell · GitHub
[go: up one dir, main page]
Skip to content

Avoid unnecessary StringCollection allocation #15832

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
Jul 27, 2021
Merged

Avoid unnecessary StringCollection allocation #15832

merged 1 commit into from
Jul 27, 2021

Conversation

xtqqczze
Copy link
Contributor

Avoid unnecessary StringCollection allocation in Microsoft.PowerShell.Commands.Internal.Format.OutCommandInner.ListOutputContext.

Split from #13431 (8bcb8ca).

@ghost ghost assigned iSazonov Jul 27, 2021
@xtqqczze
Copy link
Contributor Author

PowerShell-CI-static-analysis "Verify Markdown Links" failures appear transient.

@iSazonov iSazonov requested a review from rjmholt July 27, 2021 16:21
@iSazonov iSazonov added the CL-CodeCleanup Indicates that a PR should be marked as a Code Cleanup change in the Change Log label Jul 27, 2021
@rjmholt rjmholt merged commit 3733a0b into PowerShell:master Jul 27, 2021
@iSazonov iSazonov added this to the 7.2.0-preview.9 milestone Jul 28, 2021
@iSazonov iSazonov assigned rjmholt and unassigned iSazonov Jul 28, 2021
@xtqqczze xtqqczze deleted the StringCollection-to-array branch July 28, 2021 14:58
@ghost
Copy link
ghost commented Aug 23, 2021

🎉v7.2.0-preview.9 has been released which incorporates this pull request.:tada:

Handy links:

@ghost
Copy link
ghost commented Sep 28, 2021

🎉v7.2.0-preview.10 has been released which incorporates this pull request.:tada:

Handy links:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rjmholt rjmholt rjmholt approved these changes

@iSazonov iSazonov iSazonov approved these changes

Assignees

@rjmholt rjmholt

Labels
CL-CodeCleanup Indicates that a PR should be marked as a Code Cleanup change in the Change Log
Projects
None yet
Milestone
7.2.0-preview.9
Development

Successfully merging this pull request may close these issues.
3 participants
@xtqqczze @rjmholt @iSazonov
0