Skip to content

Use Qt's QFile abstraction for the Platform.h file I/O functions - #2646

Merged
Arisotura merged 2 commits into
melonDS-emu:masterfrom
nadiaholmquist:qfile-functions
May 6, 2026
Merged

Use Qt's QFile abstraction for the Platform.h file I/O functions#2646
Arisotura merged 2 commits into
melonDS-emu:masterfrom
nadiaholmquist:qfile-functions

Conversation

@nadiaholmquist

Copy link
Copy Markdown
Collaborator

melonDS is currently rather pointlessly opening files through Qt and then duplicating and reopening the file with the stdio file functions, there's not really any reason to do this anymore since the core no longer works with plain FILE*s, and since we're using Qt for the frontend, I feel it would be more appropriate to also have Qt's file abstractions handle this.

Avoiding this hack also means we don't need to worry about Windows's stdio weirdness and might hopefully also improve robustness of melonDS's file operations a bit.

@Arisotura
Arisotura merged commit 1609a0f into melonDS-emu:master May 6, 2026
10 checks passed
Zection6V pushed a commit to ag-advania/melonPrimeDS that referenced this pull request Aug 13, 2026
…onDS-emu#2646)

* Use Qt's QFile abstraction for the Platform.h file I/O functions

* forgot to reimplement FileWriteFormatted
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