[PR #622] Qt/GameListSettings: Use native path separators #68

New Issue

claunia · 2026-01-29T19:06:16Z

claunia commented

2026-01-29 19:06:16 +00:00

Original Pull Request: https://github.com/stenzek/duckstation/pull/622

State: closed
Merged: Yes

QFileDialog::getExistingDirectory() returns paths with forward slash by default. This path string eventually gets passed to FileSystem::FindFiles() which constructs the rest of the path string using native path separators. This means that for example if on Windows we had added C:/path/to/game non-recursively and C:/path/to recursively through Qt, we would find C:/path/to/game\x.cue and C:/path/to\game\x.cue, resulting in the same game appearing twice on the game list.

**Original Pull Request:** https://github.com/stenzek/duckstation/pull/622 **State:** closed **Merged:** Yes --- `QFileDialog::getExistingDirectory()` returns paths with forward slash by default. This path string eventually gets passed to `FileSystem::FindFiles()` which constructs the rest of the path string using native path separators. This means that for example if on Windows we had added `C:/path/to/game` non-recursively and `C:/path/to` recursively through Qt, we would find `C:/path/to/game\x.cue` and `C:/path/to\game\x.cue`, resulting in the same game appearing twice on the game list.

claunia added the pull-request label 2026-01-29 19:06:16 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/duckstation#68