Skip to content

Get the benchmarks working again.#2886

Merged
JimBobSquarePants merged 2 commits into
mainfrom
js/fix-benchmarks
Feb 10, 2025
Merged

Get the benchmarks working again.#2886
JimBobSquarePants merged 2 commits into
mainfrom
js/fix-benchmarks

Conversation

@JimBobSquarePants

@JimBobSquarePants JimBobSquarePants commented Feb 10, 2025

Copy link
Copy Markdown
Member

Prerequisites

  • I have written a descriptive pull-request title
  • I have verified that there are no overlapping pull-requests open
  • I have verified that I am following the existing coding patterns and practice as demonstrated in the repository. These follow strict Stylecop rules 👮.
  • I have provided test coverage for my change (where applicable)

Description

Who knows why the Benchmarks suddenly stopped working but I couldn't get them to build as part of the BenchmarkDotNet process despite being able to build it just fine in VS. I cleaned up several warnings and suppressed others.


PixelConverter.FromRgba32.ToArgb32(source, dest);
}
// Commenting this out because for some reason MSBuild is showing error CS0029: Cannot implicitly convert type 'System.ReadOnlySpan<byte>' to 'System.Span<byte>'

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This is just plain odd. The code is correct but CS0029 still throws.

error CS0029: Cannot implicitly convert type 'System.ReadOnlySpan<byte>' to 'System.Span<byte>'

@JimBobSquarePants JimBobSquarePants merged commit e925b77 into main Feb 10, 2025
@JimBobSquarePants JimBobSquarePants deleted the js/fix-benchmarks branch February 10, 2025 05:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant