Skip to content

Fix ThunkGenerator - #84390

Merged
jkotas merged 2 commits into
dotnet:mainfrom
MichalPetryka:patch-6
Apr 6, 2023
Merged

Fix ThunkGenerator#84390
jkotas merged 2 commits into
dotnet:mainfrom
MichalPetryka:patch-6

Conversation

@MichalPetryka

Copy link
Copy Markdown
Contributor

Fixes:

 error CA1859: Change type of parameter 'tr' from 'System.IO.TextReader' to 'System.IO.StreamReader' for improved performance
error CA1859: Change return type of method 'ParseInput' from 'System.Collections.Generic.IEnumerable<Thunkerator.FunctionDecl>' to 'System.Collections.ObjectModel.ReadOnlyCollection<Thunkerator.FunctionDecl>' for improved performance

Fixes:
```
error CA1859: Change type of parameter 'tr' from 'System.IO.TextReader' to 'System.IO.StreamReader' for improved performance
error CA1859: Change return type of method 'ParseInput' from 'System.Collections.Generic.IEnumerable<Thunkerator.FunctionDecl>' to 'System.Collections.ObjectModel.ReadOnlyCollection<Thunkerator.FunctionDecl>' for improved performance
```
@ghostghost added area-ReadyToRun community-contribution Indicates that the PR has been added by a community member labels Apr 6, 2023

@MichalStrehovskyMichalStrehovsky 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.

Thank you!

@EgorBo

EgorBo commented Apr 6, 2023

Copy link
Copy Markdown
Member

till the next build break (when new analyzers are added), it needs to be executed as part of CI - it's not the first time it's fixed separately because of that

@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 merged commit f0fbcc2 into dotnet:mainApr 6, 2023
@ghostghost locked as resolved and limited conversation to collaborators May 6, 2023
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-ReadyToRuncommunity-contributionIndicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@MichalPetryka@EgorBo@jkotas@MichalStrehovsky