Skip to content

Remove PEAssembly::UrlToPath and PEAssembly::FindLastPathSeparator#131359

Open
elinor-fung with Copilot wants to merge 1 commit into
mainfrom
copilot/remove-peassembly-urltopath
Open

Remove PEAssembly::UrlToPath and PEAssembly::FindLastPathSeparator#131359
elinor-fung with Copilot wants to merge 1 commit into
mainfrom
copilot/remove-peassembly-urltopath

Conversation

Copilot AI commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

Remove two dead/thin static helpers from PEAssembly:

  • UrlToPath — no callers; deleted outright.
  • FindLastPathSeparator — was a one-liner wrapping SString::FindBack; inlined at its two call sites in nativeimage.cpp and nativelibrary.cpp.

cc @dotnet/appmodel @AaronRobinsonMSFT

Co-authored-by: elinor-fung <47805090+elinor-fung@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because there is no eligible user to bill. To allow Copilot reviews on bot-authored pull requests, enable direct organization billing in your organization's Copilot settings.

@elinor-fung
elinor-fung marked this pull request as ready for review July 25, 2026 01:19
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 3 pipeline(s).
13 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

1 similar comment
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 3 pipeline(s).
13 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @agocke
See info in area-owners.md if you want to be subscribed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

4 participants