Skip to content

Fix bitrot in System.Linq.Expressions#38438

Merged
stephentoub merged 1 commit into
dotnet:masterfrom
MichalStrehovsky:bitrot
Jun 26, 2020
Merged

Fix bitrot in System.Linq.Expressions#38438
stephentoub merged 1 commit into
dotnet:masterfrom
MichalStrehovsky:bitrot

Conversation

@MichalStrehovsky

Copy link
Copy Markdown
Member

System.Linq.Expressions is not getting compiled without FEATURE_COMPILE so it started rotting.

System.Linq.Expressions is not getting compiled without FEATURE_COMPILE so it started rotting.

* Nullable annotations
* Fix break introduced in dotnet#32768
@ghost

ghost commented Jun 26, 2020

Copy link
Copy Markdown

Tagging subscribers to this area: @cston
Notify danmosemsft if you want to be subscribed.

@stephentoub stephentoub 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. I assume you found this because we're going to compile without this flag for... wasm?

@MichalStrehovsky

Copy link
Copy Markdown
Member Author

Thanks. I assume you found this because we're going to compile without this flag for... wasm?

Yup, I filed #38439 to consider it and I needed the size numbers for this configuration to give my motivation for this.

@stephentoub

Copy link
Copy Markdown
Member

needed the size numbers

Out of curiosity, what's the impact?

@stephentoub

stephentoub commented Jun 26, 2020

Copy link
Copy Markdown
Member

Nevermind, I see the numbers in that issue.

@stephentoub stephentoub merged commit a87ebc5 into dotnet:master Jun 26, 2020
@MichalStrehovsky MichalStrehovsky deleted the bitrot branch June 26, 2020 13:06
@ghost ghost locked as resolved and limited conversation to collaborators Dec 8, 2020
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.

3 participants