Skip to content

Remove ArchFile - #8389

Merged
nbolton merged 3 commits into
masterfrom
rmArchFile
Mar 25, 2025
Merged

Remove ArchFile#8389
nbolton merged 3 commits into
masterfrom
rmArchFile

Conversation

@sithlord48

@sithlord48 sithlord48 commented Mar 22, 2025

Copy link
Copy Markdown
Member

One Step closer to being rid of ARCH This removes the ARCH_FILE class

  • remove unused lib/arch/multibyte.h
  • replace ARCH->getBaseFilename with QFileInfo::fileName()
  • move the only use of ARCH->getInstalledDirectory to MSWindowsWatchdog
  • remove ARCH_FILE
    • rm lib/arch/IArchFile.h
    • rm lib/arch/unix/ArchFileUnix.[cpp|h]
    • rm lib/arch/win32/ArchFileWindows.[cpp|h]

@sithlord48
sithlord48 requested a review from nbolton March 22, 2025 13:07
Comment thread src/lib/deskflow/ArgParser.cpp Fixed
Comment thread src/lib/deskflow/ClientApp.cpp Fixed
Comment thread src/lib/deskflow/ServerApp.cpp Fixed
@sithlord48 sithlord48 changed the title Rm arch file Remove ArchFile Mar 22, 2025
@sithlord48
sithlord48 force-pushed the rmArchFile branch 2 times, most recently from 2210456 to abb3e29 Compare March 24, 2025 12:07

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

Looks good but I want to test on Windows before approving. Hopefully, I can do this later today or tomorrow morning when I'm back in the office.

@nbolton
nbolton merged commit 31e95ad into master Mar 25, 2025
@nbolton
nbolton deleted the rmArchFile branch March 25, 2025 08:39
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.

3 participants