Skip to content

Remove unused items in ArchMiscWindows - #8392

Merged
nbolton merged 11 commits into
masterfrom
cleanupArchMiscWindows
Mar 24, 2025
Merged

Remove unused items in ArchMiscWindows#8392
nbolton merged 11 commits into
masterfrom
cleanupArchMiscWindows

Conversation

@sithlord48

@sithlord48 sithlord48 commented Mar 22, 2025

Copy link
Copy Markdown
Member

Remove a bunch of unused methods from ArchMiscWindows

  • remove unused ArchMiscWindows::hasValue
  • remove unused ArchMiscWindows::setIcons
  • remove unused ArchMiscWindows::getIcons
  • remove unused ArchMiscWindows::deleteValue
  • remove unused ArchMiscWindows::deleteKeyTree
  • remove unused ArchMiscWindows::setValueBinary
  • remove unused ArchMiscWindows::readValueBinary
  • remove unused ArchMiscWindows::addDialog
  • remove unused ArchMiscWindows::removeDialog
  • remove unused ArchMiscWindows::s_dialog
  • remove unused ArchMiscWindows::cleanup
  • remove unused ArchMiscWindows::processDialog

@sithlord48
sithlord48 requested a review from nbolton March 22, 2025 23:38
@sithlord48 sithlord48 changed the title Cleanup arch misc windows Remove unused items in ArchMiscWindows Mar 22, 2025
@sithlord48 sithlord48 changed the title Remove unused items in ArchMiscWindows Remove unused items in ArchMiscWindows Mar 22, 2025
@sithlord48
sithlord48 force-pushed the cleanupArchMiscWindows branch from d0c210e to 1d502b5 Compare March 24, 2025 11:58

@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.

Can't test on Windows right now but should be find if CI can build.

@nbolton
nbolton enabled auto-merge (rebase) March 24, 2025 12:05
@sithlord48
sithlord48 force-pushed the cleanupArchMiscWindows branch from 1d502b5 to 41a1300 Compare March 24, 2025 12:07
@nbolton
nbolton merged commit c1a7b83 into master Mar 24, 2025
@nbolton
nbolton deleted the cleanupArchMiscWindows branch March 24, 2025 12:16
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.

2 participants