diff options
author | Sergey Bugaev <bugaevc@serenityos.org> | 2020-05-26 14:52:44 +0300 |
---|---|---|
committer | Andreas Kling <kling@serenityos.org> | 2020-05-26 14:35:10 +0200 |
commit | 602c3fdb3a0975418886e32cf9cc53b45d2f8964 (patch) | |
tree | 079b3ebdaf7db517625496e35acf1158adacad77 /Applications/FileManager | |
parent | f746bbda174d914d5de9379084a6fb2095c58d68 (diff) | |
download | serenity-602c3fdb3a0975418886e32cf9cc53b45d2f8964.zip |
AK: Rename FileSystemPath -> LexicalPath
And move canonicalized_path() to a static method on LexicalPath.
This is to make it clear that FileSystemPath/canonicalized_path() only
perform *lexical* canonicalization.
Diffstat (limited to 'Applications/FileManager')
-rw-r--r-- | Applications/FileManager/DirectoryView.cpp | 1 | ||||
-rw-r--r-- | Applications/FileManager/FileUtils.cpp | 18 | ||||
-rw-r--r-- | Applications/FileManager/PropertiesDialog.cpp | 9 | ||||
-rw-r--r-- | Applications/FileManager/PropertiesDialog.h | 1 | ||||
-rw-r--r-- | Applications/FileManager/main.cpp | 14 |
5 files changed, 21 insertions, 22 deletions
diff --git a/Applications/FileManager/DirectoryView.cpp b/Applications/FileManager/DirectoryView.cpp index 2efa86825f..4873277dca 100644 --- a/Applications/FileManager/DirectoryView.cpp +++ b/Applications/FileManager/DirectoryView.cpp @@ -25,7 +25,6 @@ */ #include "DirectoryView.h" -#include <AK/FileSystemPath.h> #include <AK/NumberFormat.h> #include <AK/StringBuilder.h> #include <AK/URL.h> diff --git a/Applications/FileManager/FileUtils.cpp b/Applications/FileManager/FileUtils.cpp index 82580e1d2b..20187415d7 100644 --- a/Applications/FileManager/FileUtils.cpp +++ b/Applications/FileManager/FileUtils.cpp @@ -25,7 +25,7 @@ */ #include "FileUtils.h" -#include <AK/FileSystemPath.h> +#include <AK/LexicalPath.h> #include <AK/StringBuilder.h> #include <LibCore/DirIterator.h> #include <stdio.h> @@ -134,7 +134,7 @@ bool copy_file(const String& src_path, const String& dst_path, const struct stat if (errno != EISDIR) { return false; } - auto dst_dir_path = String::format("%s/%s", dst_path.characters(), FileSystemPath(src_path).basename().characters()); + auto dst_dir_path = String::format("%s/%s", dst_path.characters(), LexicalPath(src_path).basename().characters()); dst_fd = creat(dst_dir_path.characters(), 0666); if (dst_fd < 0) { return false; @@ -186,21 +186,21 @@ String get_duplicate_name(const String& path, int duplicate_count) if (duplicate_count == 0) { return path; } - FileSystemPath fsp(path); + LexicalPath lexical_path(path); StringBuilder duplicated_name; duplicated_name.append('/'); - for (size_t i = 0; i < fsp.parts().size() - 1; ++i) { - duplicated_name.appendf("%s/", fsp.parts()[i].characters()); + for (size_t i = 0; i < lexical_path.parts().size() - 1; ++i) { + duplicated_name.appendf("%s/", lexical_path.parts()[i].characters()); } auto prev_duplicate_tag = String::format("(%d)", duplicate_count); - auto title = fsp.title(); + auto title = lexical_path.title(); if (title.ends_with(prev_duplicate_tag)) { // remove the previous duplicate tag "(n)" so we can add a new tag. title = title.substring(0, title.length() - prev_duplicate_tag.length()); } - duplicated_name.appendf("%s (%d)", fsp.title().characters(), duplicate_count); - if (!fsp.extension().is_empty()) { - duplicated_name.appendf(".%s", fsp.extension().characters()); + duplicated_name.appendf("%s (%d)", lexical_path.title().characters(), duplicate_count); + if (!lexical_path.extension().is_empty()) { + duplicated_name.appendf(".%s", lexical_path.extension().characters()); } return duplicated_name.build(); } diff --git a/Applications/FileManager/PropertiesDialog.cpp b/Applications/FileManager/PropertiesDialog.cpp index 0813e5fbd0..8caa182dd5 100644 --- a/Applications/FileManager/PropertiesDialog.cpp +++ b/Applications/FileManager/PropertiesDialog.cpp @@ -25,6 +25,7 @@ */ #include "PropertiesDialog.h" +#include <AK/LexicalPath.h> #include <AK/StringBuilder.h> #include <LibGUI/BoxLayout.h> #include <LibGUI/CheckBox.h> @@ -42,8 +43,8 @@ PropertiesDialog::PropertiesDialog(GUI::FileSystemModel& model, String path, boo : Dialog(parent_window) , m_model(model) { - auto file_path = FileSystemPath(path); - ASSERT(file_path.is_valid()); + auto lexical_path = LexicalPath(path); + ASSERT(lexical_path.is_valid()); auto& main_widget = set_main_widget<GUI::Widget>(); main_widget.set_layout<GUI::VerticalBoxLayout>(); @@ -72,8 +73,8 @@ PropertiesDialog::PropertiesDialog(GUI::FileSystemModel& model, String path, boo m_icon->set_size_policy(GUI::SizePolicy::Fixed, GUI::SizePolicy::Fixed); m_icon->set_preferred_size(32, 32); - m_name = file_path.basename(); - m_path = file_path.string(); + m_name = lexical_path.basename(); + m_path = lexical_path.string(); m_name_box = file_container.add<GUI::TextBox>(); m_name_box->set_size_policy(GUI::SizePolicy::Fill, GUI::SizePolicy::Fixed); diff --git a/Applications/FileManager/PropertiesDialog.h b/Applications/FileManager/PropertiesDialog.h index 6bb5621936..3ed9554194 100644 --- a/Applications/FileManager/PropertiesDialog.h +++ b/Applications/FileManager/PropertiesDialog.h @@ -26,7 +26,6 @@ #pragma once -#include <AK/FileSystemPath.h> #include <LibCore/File.h> #include <LibGUI/Button.h> #include <LibGUI/Dialog.h> diff --git a/Applications/FileManager/main.cpp b/Applications/FileManager/main.cpp index 6ecdb03e9f..c2d2547dc9 100644 --- a/Applications/FileManager/main.cpp +++ b/Applications/FileManager/main.cpp @@ -27,7 +27,7 @@ #include "DirectoryView.h" #include "FileUtils.h" #include "PropertiesDialog.h" -#include <AK/FileSystemPath.h> +#include <AK/LexicalPath.h> #include <AK/StringBuilder.h> #include <AK/URL.h> #include <LibCore/ConfigFile.h> @@ -165,7 +165,7 @@ int run_in_desktop_mode(RefPtr<Core::ConfigFile> config, String initial_location auto mkdir_action = GUI::Action::create("New directory...", {}, Gfx::Bitmap::load_from_file("/res/icons/16x16/mkdir.png"), [&](const GUI::Action&) { auto input_box = GUI::InputBox::construct("Enter name:", "New directory", window); if (input_box->exec() == GUI::InputBox::ExecOK && !input_box->text_value().is_empty()) { - auto new_dir_path = canonicalized_path( + auto new_dir_path = LexicalPath::canonicalized_path( String::format("%s/%s", model->root_path().characters(), input_box->text_value().characters())); @@ -179,7 +179,7 @@ int run_in_desktop_mode(RefPtr<Core::ConfigFile> config, String initial_location auto touch_action = GUI::Action::create("New file...", {}, Gfx::Bitmap::load_from_file("/res/icons/16x16/new.png"), [&](const GUI::Action&) { auto input_box = GUI::InputBox::construct("Enter name:", "New file", window); if (input_box->exec() == GUI::InputBox::ExecOK && !input_box->text_value().is_empty()) { - auto new_file_path = canonicalized_path( + auto new_file_path = LexicalPath::canonicalized_path( String::format("%s/%s", model->root_path().characters(), input_box->text_value().characters())); @@ -323,7 +323,7 @@ int run_in_windowed_mode(RefPtr<Core::ConfigFile> config, String initial_locatio auto mkdir_action = GUI::Action::create("New directory...", { Mod_Ctrl | Mod_Shift, Key_N }, Gfx::Bitmap::load_from_file("/res/icons/16x16/mkdir.png"), [&](const GUI::Action&) { auto input_box = GUI::InputBox::construct("Enter name:", "New directory", window); if (input_box->exec() == GUI::InputBox::ExecOK && !input_box->text_value().is_empty()) { - auto new_dir_path = canonicalized_path( + auto new_dir_path = LexicalPath::canonicalized_path( String::format("%s/%s", directory_view.path().characters(), input_box->text_value().characters())); @@ -442,7 +442,7 @@ int run_in_windowed_mode(RefPtr<Core::ConfigFile> config, String initial_locatio selected = selected_file_paths(); } else { path = directories_model->full_path(tree_view.selection().first()); - container_dir_path = FileSystemPath(path).basename(); + container_dir_path = LexicalPath(path).basename(); selected = tree_view_selected_file_paths(); } @@ -510,7 +510,7 @@ int run_in_windowed_mode(RefPtr<Core::ConfigFile> config, String initial_locatio String message; if (paths.size() == 1) { - message = String::format("Really delete %s?", FileSystemPath(paths[0]).basename().characters()); + message = String::format("Really delete %s?", LexicalPath(paths[0]).basename().characters()); } else { message = String::format("Really delete %d files?", paths.size()); } @@ -791,7 +791,7 @@ int run_in_windowed_mode(RefPtr<Core::ConfigFile> config, String initial_locatio continue; auto new_path = String::format("%s/%s", target_node.full_path(directory_view.model()).characters(), - FileSystemPath(url_to_copy.path()).basename().characters()); + LexicalPath(url_to_copy.path()).basename().characters()); if (url_to_copy.path() == new_path) continue; |