diff options
author | Lioncash <mathew1800@gmail.com> | 2018-07-21 15:52:42 -0400 |
---|---|---|
committer | Lioncash <mathew1800@gmail.com> | 2018-07-21 16:21:19 -0400 |
commit | d66b43dadfac1e9324fee48e97361e2f858f8af5 (patch) | |
tree | 6007104127ffaa62cb5cb94f5c47fadf4c192fb8 /src/common/common_paths.h | |
parent | 0f20fa5a1e179bdad07fe1e0e58243d1927274fd (diff) |
file_util: Use an enum class for GetUserPath()
Instead of using an unsigned int as a parameter and expecting a user to
always pass in the correct values, we can just convert the enum into an
enum class and use that type as the parameter type instead, which makes
the interface more type safe.
We also get rid of the bookkeeping "NUM_" element in the enum by just
using an unordered map. This function is generally low-frequency in
terms of calls (and I'd hope so, considering otherwise would mean we're
slamming the disk with IO all the time) so I'd consider this acceptable
in this case.
Diffstat (limited to 'src/common/common_paths.h')
-rw-r--r-- | src/common/common_paths.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/common/common_paths.h b/src/common/common_paths.h index 9bf3efaf27..6799a357ae 100644 --- a/src/common/common_paths.h +++ b/src/common/common_paths.h @@ -26,7 +26,7 @@ #define USA_DIR "USA" #define JAP_DIR "JAP" -// Subdirs in the User dir returned by GetUserPath(D_USER_IDX) +// Subdirs in the User dir returned by GetUserPath(UserPath::UserDir) #define CONFIG_DIR "config" #define CACHE_DIR "cache" #define SDMC_DIR "sdmc" @@ -35,11 +35,11 @@ #define LOG_DIR "log" // Filenames -// Files in the directory returned by GetUserPath(D_CONFIG_IDX) +// Files in the directory returned by GetUserPath(UserPath::ConfigDir) #define EMU_CONFIG "emu.ini" #define DEBUGGER_CONFIG "debugger.ini" #define LOGGER_CONFIG "logger.ini" -// Files in the directory returned by GetUserPath(D_LOGS_IDX) +// Files in the directory returned by GetUserPath(UserPath::LogDir) #define LOG_FILE "yuzu_log.txt" // Sys files |