New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Preallocate dictionary size in ToDictionaryWithConflictLog #19907
Conversation
I suspect the build failure will be solved by #19902 |
Only the build compiling with Mono is failing:
Does Mono not contain this method? |
No, it was added in net 6.
…On Sun, 30 Jan 2022, 11:38 abcdefg30, ***@***.***> wrote:
I suspect the build failure will be solved by #19902
<#19902>
Only the build compiling with Mono is failing:
Exts.cs(410,11): error CS1061: 'IEnumerable' does not contain a definition
for 'TryGetNonEnumeratedCount' and no accessible extension method
'TryGetNonEnumeratedCount' accepting a first argument of type 'IEnumerable'
could be found (are you missing a using directive or an assembly
reference?) [/home/runner/work/OpenRA/OpenRA/OpenRA.Game/OpenRA.Game.csproj]
Does Mono not contain this method?
—
Reply to this email directly, view it on GitHub
<#19907 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AESI7GA7RYB3UI3SGYDFVR3UYUIJ7ANCNFSM5NDJM3YQ>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
Then we might need to add a Mono only path that uses the old code. |
edae15c
to
dc15b62
Compare
Switched to a cast on |
Makes sense 👍 |
dc15b62
to
bcc5a8a
Compare
This saves some allocations in MiniYaml parsing on startup, but with also benefit other callers of the method.