Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Use SequenceEqual in win32 GetFullPath - #10706

#10706
Merged
YuliiaKovalova merged 2 commits into
dotnet:maindotnet/msbuild:mainfrom
rainersigwald:sequenceequal-windows-getfullpathrainersigwald/msbuild:sequenceequal-windows-getfullpathCopy head branch name to clipboard
Oct 2, 2024
Merged

Use SequenceEqual in win32 GetFullPath#10706
YuliiaKovalova merged 2 commits into
dotnet:maindotnet/msbuild:mainfrom
rainersigwald:sequenceequal-windows-getfullpathrainersigwald/msbuild:sequenceequal-windows-getfullpathCopy head branch name to clipboard

Conversation

@rainersigwald

Copy link
Copy Markdown
Member

This should be slightly more efficient since it's vectorizable.

This should be slightly more efficient since it's vectorizable.
@MichalPavlik

Copy link
Copy Markdown
Member

Some constructs are not supported in net35 (MSBuildTaskHost uses this file). I'm afraid a new #if will be required :(

@rainersigwald

Copy link
Copy Markdown
Member Author

Fixed by replacing the body of the existing function, under an ifdef.

*extremely deep sigh*

@YuliiaKovalova
YuliiaKovalova merged commit c034fa1 into dotnet:main Oct 2, 2024
@rainersigwald
rainersigwald deleted the sequenceequal-windows-getfullpath branch October 2, 2024 19:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.