diff options
Diffstat (limited to 'src')
-rw-r--r-- | src/CommandLine.cxx | 2 | ||||
-rw-r--r-- | src/Main.cxx | 4 | ||||
-rw-r--r-- | src/Mapper.cxx | 4 | ||||
-rw-r--r-- | src/PlaylistFile.cxx | 2 | ||||
-rw-r--r-- | src/command/FileCommands.cxx | 3 | ||||
-rw-r--r-- | src/config/Path.cxx | 2 | ||||
-rw-r--r-- | src/db/Configured.cxx | 3 | ||||
-rw-r--r-- | src/db/plugins/simple/SimpleDatabasePlugin.cxx | 3 | ||||
-rw-r--r-- | src/db/update/InotifyUpdate.cxx | 10 | ||||
-rw-r--r-- | src/fs/CheckFile.cxx | 3 | ||||
-rw-r--r-- | src/fs/Path.hxx | 10 | ||||
-rw-r--r-- | src/fs/Path2.cxx | 8 | ||||
-rw-r--r-- | src/fs/StandardDirectory.cxx | 13 | ||||
-rw-r--r-- | src/storage/plugins/LocalStorage.cxx | 5 |
14 files changed, 40 insertions, 32 deletions
diff --git a/src/CommandLine.cxx b/src/CommandLine.cxx index ecfae241e..f1d6d0020 100644 --- a/src/CommandLine.cxx +++ b/src/CommandLine.cxx @@ -310,7 +310,7 @@ bool ConfigLoader::TryFile(const AllocatedPath &base_path, Path path) { if (base_path.IsNull()) return false; - auto full_path = AllocatedPath::Build(base_path, path); + auto full_path = base_path / path; return TryFile(full_path); } diff --git a/src/Main.cxx b/src/Main.cxx index a489c652a..536f2ac90 100644 --- a/src/Main.cxx +++ b/src/Main.cxx @@ -260,7 +260,7 @@ glue_state_file_init() if (cache_dir.IsNull()) return; - path_fs = AllocatedPath::Build(cache_dir, "state"); + path_fs = cache_dir / Path::FromFS("state"); #else return; #endif @@ -486,7 +486,7 @@ try { const auto sdcard = Environment::getExternalStorageDirectory(); if (!sdcard.IsNull()) { const auto config_path = - AllocatedPath::Build(sdcard, "mpd.conf"); + sdcard / Path::FromFS("mpd.conf"); if (FileExists(config_path)) ReadConfigFile(config_path); } diff --git a/src/Mapper.cxx b/src/Mapper.cxx index 089469ea3..bb0387060 100644 --- a/src/Mapper.cxx +++ b/src/Mapper.cxx @@ -82,7 +82,7 @@ map_uri_fs(const char *uri) noexcept if (uri_fs.IsNull()) return nullptr; - return AllocatedPath::Build(music_dir_fs, uri_fs); + return music_dir_fs / uri_fs; } std::string @@ -128,5 +128,5 @@ map_spl_utf8_to_fs(const char *name) noexcept if (filename_fs.IsNull()) return nullptr; - return AllocatedPath::Build(playlist_dir_fs, filename_fs); + return playlist_dir_fs / filename_fs; } diff --git a/src/PlaylistFile.cxx b/src/PlaylistFile.cxx index 101e9ecec..a1b83eb41 100644 --- a/src/PlaylistFile.cxx +++ b/src/PlaylistFile.cxx @@ -138,7 +138,7 @@ LoadPlaylistFileInfo(PlaylistInfo &info, return false; FileInfo fi; - if (!GetFileInfo(AllocatedPath::Build(parent_path_fs, name_fs), fi) || + if (!GetFileInfo(parent_path_fs / name_fs, fi) || !fi.IsRegular()) return false; diff --git a/src/command/FileCommands.cxx b/src/command/FileCommands.cxx index c6b54cee8..8a2167789 100644 --- a/src/command/FileCommands.cxx +++ b/src/command/FileCommands.cxx @@ -81,8 +81,7 @@ handle_listfiles_local(Response &r, Path path_fs) if (name_utf8.empty()) continue; - const AllocatedPath full_fs = - AllocatedPath::Build(path_fs, name_fs); + const auto full_fs = path_fs / name_fs; FileInfo fi; if (!GetFileInfo(full_fs, fi, false)) continue; diff --git a/src/config/Path.cxx b/src/config/Path.cxx index 32d23c9df..e5e4256ae 100644 --- a/src/config/Path.cxx +++ b/src/config/Path.cxx @@ -113,7 +113,7 @@ ParsePath(const char *path) if (path2.IsNull()) return nullptr; - return AllocatedPath::Build(home, path2); + return home / path2; } else if (!PathTraitsUTF8::IsAbsolute(path)) { throw FormatRuntimeError("not an absolute path: %s", path); } else { diff --git a/src/db/Configured.cxx b/src/db/Configured.cxx index 79b28197b..fc0c7d5dd 100644 --- a/src/db/Configured.cxx +++ b/src/db/Configured.cxx @@ -53,8 +53,7 @@ CreateConfiguredDatabase(EventLoop &main_event_loop, EventLoop &io_event_loop, if (cache_dir.IsNull()) return nullptr; - const auto db_file = AllocatedPath::Build(cache_dir, - PATH_LITERAL("mpd.db")); + const auto db_file = cache_dir / Path::FromFS(PATH_LITERAL("mpd.db")); const auto db_file_utf8 = db_file.ToUTF8(); if (db_file_utf8.empty()) return nullptr; diff --git a/src/db/plugins/simple/SimpleDatabasePlugin.cxx b/src/db/plugins/simple/SimpleDatabasePlugin.cxx index 6ea2bf9bd..94510110c 100644 --- a/src/db/plugins/simple/SimpleDatabasePlugin.cxx +++ b/src/db/plugins/simple/SimpleDatabasePlugin.cxx @@ -428,8 +428,7 @@ SimpleDatabase::Mount(const char *local_uri, const char *storage_uri) #ifndef ENABLE_ZLIB constexpr bool compress = false; #endif - auto db = new SimpleDatabase(AllocatedPath::Build(cache_path, - name_fs.c_str()), + auto db = new SimpleDatabase(cache_path / name_fs, compress); try { db->Open(); diff --git a/src/db/update/InotifyUpdate.cxx b/src/db/update/InotifyUpdate.cxx index e1a1e9dbd..d0ea2daec 100644 --- a/src/db/update/InotifyUpdate.cxx +++ b/src/db/update/InotifyUpdate.cxx @@ -141,7 +141,7 @@ WatchDirectory::GetUriFS() const noexcept if (uri.IsNull()) return name; - return AllocatedPath::Build(uri, name); + return uri / name; } /* we don't look at "." / ".." nor files with newlines in their name */ @@ -181,8 +181,8 @@ recursive_watch_subdirectories(WatchDirectory *directory, if (skip_path(ent->d_name)) continue; - const auto child_path_fs = - AllocatedPath::Build(path_fs, ent->d_name); + const auto name_fs = Path::FromFS(ent->d_name); + const auto child_path_fs = path_fs / name_fs; FileInfo fi; try { @@ -211,7 +211,7 @@ recursive_watch_subdirectories(WatchDirectory *directory, continue; directory->children.emplace_front(directory, - AllocatedPath::FromFS(ent->d_name), + name_fs, ret); child = &directory->children.front(); @@ -262,7 +262,7 @@ mpd_inotify_callback(int wd, unsigned mask, const auto path_fs = uri_fs.IsNull() ? root - : AllocatedPath::Build(root, uri_fs.c_str()); + : (root / uri_fs); recursive_watch_subdirectories(directory, path_fs, directory->GetDepth()); diff --git a/src/fs/CheckFile.cxx b/src/fs/CheckFile.cxx index 67578e182..189aabfe6 100644 --- a/src/fs/CheckFile.cxx +++ b/src/fs/CheckFile.cxx @@ -43,8 +43,7 @@ try { #ifndef _WIN32 try { - const auto x = AllocatedPath::Build(path_fs, - PathTraitsFS::CURRENT_DIRECTORY); + const auto x = path_fs / Path::FromFS(PathTraitsFS::CURRENT_DIRECTORY); const FileInfo fi2(x); } catch (const std::system_error &e) { if (IsAccessDenied(e)) diff --git a/src/fs/Path.hxx b/src/fs/Path.hxx index 2e7393eab..2415923d0 100644 --- a/src/fs/Path.hxx +++ b/src/fs/Path.hxx @@ -1,5 +1,5 @@ /* - * Copyright 2003-2017 The Music Player Daemon Project + * Copyright 2003-2018 The Music Player Daemon Project * http://www.musicpd.org * * This program is free software; you can redistribute it and/or modify @@ -169,4 +169,12 @@ public: const_pointer_type GetSuffix() const noexcept; }; +/** + * Concatenate two path components using the directory separator. + * + * Wrapper for AllocatedPath::Build(). + */ +AllocatedPath +operator/(Path a, Path b) noexcept; + #endif diff --git a/src/fs/Path2.cxx b/src/fs/Path2.cxx index 44410f9a2..f5f0662cf 100644 --- a/src/fs/Path2.cxx +++ b/src/fs/Path2.cxx @@ -1,5 +1,5 @@ /* - * Copyright 2003-2017 The Music Player Daemon Project + * Copyright 2003-2018 The Music Player Daemon Project * http://www.musicpd.org * * This program is free software; you can redistribute it and/or modify @@ -26,3 +26,9 @@ Path::GetDirectoryName() const noexcept { return AllocatedPath::FromFS(PathTraitsFS::GetParent(c_str())); } + +AllocatedPath +operator/(Path a, Path b) noexcept +{ + return AllocatedPath::Build(a, b); +} diff --git a/src/fs/StandardDirectory.cxx b/src/fs/StandardDirectory.cxx index 67845e39b..65d6b7a1c 100644 --- a/src/fs/StandardDirectory.cxx +++ b/src/fs/StandardDirectory.cxx @@ -183,16 +183,16 @@ ParseConfigLine(char *line, const char *dir_name, AllocatedPath &result_dir) *line_end = 0; // build the result path - const char *path = line; + const auto path_fs = Path::FromFS(line); AllocatedPath result = nullptr; if (home_relative) { auto home = GetHomeDir(); if (home.IsNull()) return true; - result = AllocatedPath::Build(home, path); + result = home / path_fs; } else { - result = AllocatedPath::FromFS(path); + result = AllocatedPath(path_fs); } if (IsValidDir(result.c_str())) { @@ -209,9 +209,8 @@ try { auto config_dir = GetUserConfigDir(); if (config_dir.IsNull()) return result; - auto dirs_file = AllocatedPath::Build(config_dir, "user-dirs.dirs"); - TextFile input(dirs_file); + TextFile input(config_dir / Path::FromFS("user-dirs.dirs")); char *line; while ((line = input.ReadLine()) != nullptr) if (ParseConfigLine(line, name, result)) @@ -237,7 +236,7 @@ GetUserConfigDir() noexcept // Check for $HOME/.config auto home = GetHomeDir(); if (!home.IsNull()) { - AllocatedPath fallback = AllocatedPath::Build(home, ".config"); + auto fallback = home / Path::FromFS(".config"); if (IsValidDir(fallback.c_str())) return fallback; } @@ -274,7 +273,7 @@ GetUserCacheDir() noexcept // Check for $HOME/.cache auto home = GetHomeDir(); if (!home.IsNull()) { - AllocatedPath fallback = AllocatedPath::Build(home, ".cache"); + auto fallback = home / Path::FromFS(".cache"); if (IsValidDir(fallback.c_str())) return fallback; } diff --git a/src/storage/plugins/LocalStorage.cxx b/src/storage/plugins/LocalStorage.cxx index 83b877648..78e5df2e4 100644 --- a/src/storage/plugins/LocalStorage.cxx +++ b/src/storage/plugins/LocalStorage.cxx @@ -114,8 +114,7 @@ LocalStorage::MapFSOrThrow(const char *uri_utf8) const if (StringIsEmpty(uri_utf8)) return base_fs; - return AllocatedPath::Build(base_fs, - AllocatedPath::FromUTF8Throw(uri_utf8)); + return base_fs / AllocatedPath::FromUTF8Throw(uri_utf8); } AllocatedPath @@ -176,7 +175,7 @@ LocalDirectoryReader::Read() noexcept StorageFileInfo LocalDirectoryReader::GetInfo(bool follow) { - return Stat(AllocatedPath::Build(base_fs, reader.GetEntry()), follow); + return Stat(base_fs / reader.GetEntry(), follow); } std::unique_ptr<Storage> |