Skip to content

Conversation

xtqqczze
Copy link
Contributor

@xtqqczze xtqqczze commented Aug 28, 2025

This change removes Interop files that are no longer part of the build. After System.Drawing.Common was removed in #83356, several related Interop files were left in the tree but are no longer compiled:

  • src/libraries/Common/src/Interop/Windows/Interop.HRESULT.cs
  • src/libraries/Common/src/Interop/Windows/Kernel32/Interop.GetSystemDefaultLCID.cs
  • src/libraries/Common/src/Interop/Windows/Kernel32/Interop.GlobalAlloc.cs
  • src/libraries/Common/src/Interop/Windows/Kernel32/Interop.GlobalFree.cs
  • src/libraries/Common/src/Interop/Windows/Kernel32/Interop.GlobalLock.cs
  • src/libraries/Common/src/Interop/Windows/Kernel32/Interop.SelectObject.cs
  • src/libraries/Common/src/Interop/Windows/Ole32/Interop.IStream.COMWrappers.cs
  • src/libraries/Common/src/Interop/Windows/Ole32/Interop.IStream.NoCOMWrappers.cs
  • src/libraries/Common/src/Interop/Windows/Ole32/Interop.STATFLAG.cs
  • src/libraries/Common/src/Interop/Windows/Ole32/Interop.STATSTG.cs
  • src/libraries/Common/src/Interop/Windows/Ole32/Interop.STGM.cs
  • src/libraries/Common/src/Interop/Windows/Ole32/Interop.STGTY.cs
  • src/libraries/Common/src/Interop/Windows/Shell32/Interop.ExtractAssociatedIcon.cs
  • src/libraries/Common/src/Interop/Windows/User32/Interop.CopyImage.cs
  • src/libraries/Common/src/Interop/Windows/User32/Interop.CreateIconFromResourceEx.cs
  • src/libraries/Common/src/Interop/Windows/User32/Interop.DestroyIcon.cs
  • src/libraries/Common/src/Interop/Windows/User32/Interop.DrawIconEx.cs
  • src/libraries/Common/src/Interop/Windows/User32/Interop.GetDC.cs
  • src/libraries/Common/src/Interop/Windows/User32/Interop.GetIconInfo.cs
  • src/libraries/Common/src/Interop/Windows/User32/Interop.GetSystemMetrics.cs
  • src/libraries/Common/src/Interop/Windows/User32/Interop.LOGFONT.cs
  • src/libraries/Common/src/Interop/Windows/User32/Interop.LoadIcon.cs
  • src/libraries/Common/src/Interop/Windows/User32/Interop.NONCLIENTMETRICS.cs
  • src/libraries/Common/src/Interop/Windows/User32/Interop.ReleaseDC.cs
  • src/libraries/Common/src/Interop/Windows/User32/Interop.WindowFromDC.cs
  • src/libraries/Common/src/Interop/Windows/Winspool/Interop.DeviceCapabilities.cs
  • src/libraries/Common/src/Interop/Windows/Winspool/Interop.DocumentProperties.cs
  • src/libraries/Common/src/Interop/Windows/Winspool/Interop.EnumPrinters.cs

cc: @ViktorHofer, @JeremyKuhne, @ericstj

After `System.Drawing.Common` was removed in dotnet#83356 there were some files left over that are now longer compiled.
@github-actions github-actions bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Aug 28, 2025
@dotnet-policy-service dotnet-policy-service bot added the community-contribution Indicates that the PR has been added by a community member label Aug 28, 2025
@xtqqczze xtqqczze changed the title Remove Interop files that are not compiled Remove Interop files that are uncompiled Aug 28, 2025
Copy link
Member

@jkotas jkotas left a comment

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 8ce75d7 into dotnet:main Aug 28, 2025
92 of 94 checks passed
@xtqqczze xtqqczze deleted the RemoveSystemDrawingCommon2 branch August 28, 2025 02:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community-contribution Indicates that the PR has been added by a community member needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants