Skip to content

Remove SString::UIndex and SString::UIterator as they are never used. - #71103

Merged
jkoritzinsky merged 1 commit into
dotnet:mainfrom
jkoritzinsky:sstring-remove-dead-code
Jun 22, 2022
Merged

Remove SString::UIndex and SString::UIterator as they are never used.#71103
jkoritzinsky merged 1 commit into
dotnet:mainfrom
jkoritzinsky:sstring-remove-dead-code

Conversation

@jkoritzinsky

@jkoritzinskyjkoritzinsky commented Jun 21, 2022

Copy link
Copy Markdown
Member

This is based on work in #69748.

@jkoritzinsky
jkoritzinsky requested a review from jkotasJune 21, 2022 23:09
@ghostghost added the area-VM-coreclr label Jun 21, 2022
jkotas
jkotas approved these changes Jun 21, 2022
View reviewed changes

@jkotasjkotas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@jkotas

jkotas commented Jun 22, 2022

Copy link
Copy Markdown
Member

Installer failure is #70969

@jkoritzinsky
jkoritzinsky merged commit 11b746a into dotnet:mainJun 22, 2022
@jkoritzinsky
jkoritzinsky deleted the sstring-remove-dead-code branch June 22, 2022 03:06
@ghostghost locked as resolved and limited conversation to collaborators Jul 22, 2022
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@jkotasjkotasjkotas approved these changes
@AaronRobinsonMSFTAaronRobinsonMSFTAaronRobinsonMSFT approved these changes

Assignees

@jkoritzinskyjkoritzinsky

Labels

area-VM-coreclr

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jkoritzinsky@jkotas@AaronRobinsonMSFT