Skip to content

Cleanup Windows classes unused code and includes - #8639

Merged
nbolton merged 2 commits into
masterfrom
rmWindowsUtils
Jun 2, 2025
Merged

Cleanup Windows classes unused code and includes#8639
nbolton merged 2 commits into
masterfrom
rmWindowsUtils

Conversation

@sithlord48

@sithlord48 sithlord48 commented May 31, 2025

Copy link
Copy Markdown
Member
  • Remove unused platform/MSWindowsUtils class
  • Remove unused windows includes

@sithlord48 sithlord48 added this to the v1.23.0 milestone May 31, 2025
@github-actions

This comment was marked as outdated.

@sithlord48
sithlord48 force-pushed the rmWindowsUtils branch 2 times, most recently from e60fa23 to 4cf1f08 Compare May 31, 2025 18:58
@sithlord48 sithlord48 changed the title Remove unused platform/MSWindowsUtils class Cleanup Windows classes unused code and includes May 31, 2025
@sithlord48 sithlord48 added 🪟 windows This issue affects Windows 🗒️ task Doesn't directly add new functionality or fix bugs in the program labels May 31, 2025
@sithlord48
sithlord48 requested a review from nbolton May 31, 2025 19:11

@nbolton nbolton left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Minor nit.

Comment thread src/lib/arch/win32/ArchDaemonWindows.cpp Outdated
@sithlord48
sithlord48 requested a review from nbolton June 2, 2025 20:27
@nbolton
nbolton merged commit a31cff0 into master Jun 2, 2025
@nbolton
nbolton deleted the rmWindowsUtils branch June 2, 2025 20:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🗒️ task Doesn't directly add new functionality or fix bugs in the program 🪟 windows This issue affects Windows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants