Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Remove use of non-generic Queue/Stack in System.Private.Xml#41142

Merged
stephentoub merged 1 commit intodotnet:masterfrom
stephentoub:xmlqueuestack
Sep 17, 2019
Merged

Remove use of non-generic Queue/Stack in System.Private.Xml#41142
stephentoub merged 1 commit intodotnet:masterfrom
stephentoub:xmlqueuestack

Conversation

@stephentoub
Copy link
Member

This is the only reason these types are brought into a trimmed default MVC app.

cc: @krwq, @jkotas

This is the only reason these types are brought into a trimmed default MVC app.
@jkotas
Copy link
Member

jkotas commented Sep 16, 2019

Are there any other references to System.Collections.NonGeneric remaining in System.Private.Xml?

@stephentoub
Copy link
Member Author

Are there any other references to System.Collections.NonGeneric remaining in System.Private.Xml?

Unfortunately, yes. Several public types derive from CollectionBase.

@stephentoub stephentoub merged commit 6347f35 into dotnet:master Sep 17, 2019
@stephentoub stephentoub deleted the xmlqueuestack branch September 17, 2019 01:04
@stephentoub stephentoub added the assembly-size Issues related to the size of assemblies, before or after trimming label Sep 19, 2019
@karelz karelz added this to the 5.0 milestone Dec 19, 2019
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
…orefx#41142)

This is the only reason these types are brought into a trimmed default MVC app.

Commit migrated from dotnet/corefx@6347f35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-System.Xml assembly-size Issues related to the size of assemblies, before or after trimming

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants