-
Notifications
You must be signed in to change notification settings - Fork 8.1k
Replace StringCollection with generic List #13431
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
Conversation
src/Microsoft.PowerShell.Commands.Diagnostics/GetCounterCommand.cs
Outdated
Show resolved
Hide resolved
This comment has been minimized.
This comment has been minimized.
src/Microsoft.PowerShell.Commands.Diagnostics/GetCounterCommand.cs
Outdated
Show resolved
Hide resolved
src/System.Management.Automation/FormatAndOutput/common/ComplexWriter.cs
Outdated
Show resolved
Hide resolved
|
@rjmholt Could we make changes in |
|
Existing formatting was left unchanged, hence Codacy issues. |
|
@iSazonov can you approve? |
|
This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days. |
161da42 to
a7a3160
Compare
|
rebased to remove merge commit |
|
@xtqqczze Please resolve merge conflicts. |
|
@xtqqczze I think it is better to split the PR to some small PRs by cmdlet/folder to get more fast progress. |
|
This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days. |
|
Please look CI fails.
and maybe split the PR by kind of changes since there is a risk of regression. |
|
This pull request has been automatically marked as stale because it has been marked as requiring author feedback but has not had any activity for 15 days. It will be closed if no further activity occurs within 10 days of this comment. |
PR Summary
StringCollectionwithList<string>.StringCollectionwithstring[]when length is known.GenerateMultiLineRowFieldas static method.Changes not made in the following:
src\System.Management.Automation\engine\LanguagePrimitives.cs: complex call hierarchysrc\Microsoft.PowerShell.Commands.Diagnostics\CounterSet.cs: parameter in public APIPR Context
Performance using the generic
List<string>is much better than using the non-genericStringCollection.See #13425 (comment) for a benchmark.
PR Checklist
.h,.cpp,.cs,.ps1and.psm1files have the correct copyright headerWIP:or[ WIP ]to the beginning of the title (theWIPbot will keep its status check atPendingwhile the prefix is present) and remove the prefix when the PR is ready.