Skip to content

Conversation

@stephentoub
Copy link
Member

Related to dotnet/wpf#6268

@ghost ghost assigned stephentoub Mar 17, 2022
@ghost
Copy link

ghost commented Mar 17, 2022

Tagging subscribers to this area: @dotnet/area-system-io-compression
See info in area-owners.md if you want to be subscribed.

Issue Details

Related to dotnet/wpf#6268

Author: stephentoub
Assignees: stephentoub
Labels:

area-System.IO.Compression

Milestone: -

@Tornhoof
Copy link
Contributor

Has cha----ter in the comment (end of line)

/// 2. If all the characters in the token are either from the remaining allowed cha----ter set.

any special meaning?, otherwise search/replace with character. This kinda looks like some older search/replace gone wrong. There is more than one hit.

@stephentoub
Copy link
Member Author

any special meaning?

Purely a guess, but I'd speculate someone meant to write "character", instead had a typo and wrote "characeter", and a well-meaning tool decided to remove a potentially sensitive substring, leaving "cha----ter". Feel free to submit a PR to clean it up.

Copy link
Member

@adamsitnik adamsitnik left a comment

Choose a reason for hiding this comment

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

LGTM, thank you @stephentoub

@stephentoub stephentoub merged commit 3b2808b into dotnet:main Mar 18, 2022
@stephentoub stephentoub deleted the packagingcontenttype branch March 18, 2022 11:49
radekdoulik pushed a commit to radekdoulik/runtime that referenced this pull request Mar 30, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Apr 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants