diff --git a/Userland/Utilities/CMakeLists.txt b/Userland/Utilities/CMakeLists.txt index 7ac661b263..29eb8f07c6 100644 --- a/Userland/Utilities/CMakeLists.txt +++ b/Userland/Utilities/CMakeLists.txt @@ -80,6 +80,7 @@ target_link_libraries(cut LibMain) target_link_libraries(date LibMain) target_link_libraries(dd LibMain) target_link_libraries(ddate LibMain) +target_link_libraries(df LibMain) target_link_libraries(diff LibDiff) target_link_libraries(disasm LibX86) target_link_libraries(dmesg LibMain) diff --git a/Userland/Utilities/df.cpp b/Userland/Utilities/df.cpp index df89529457..1a1f560d6d 100644 --- a/Userland/Utilities/df.cpp +++ b/Userland/Utilities/df.cpp @@ -4,14 +4,13 @@ * SPDX-License-Identifier: BSD-2-Clause */ -#include -#include #include #include #include #include #include #include +#include #include #include @@ -27,18 +26,14 @@ struct FileSystem { String mount_point; }; -int main(int argc, char** argv) +ErrorOr serenity_main(Main::Arguments arguments) { Core::ArgsParser args_parser; args_parser.set_general_help("Display free disk space of each partition."); args_parser.add_option(flag_human_readable, "Print human-readable sizes", "human-readable", 'h'); - args_parser.parse(argc, argv); + args_parser.parse(arguments); - auto file = Core::File::construct("/proc/df"); - if (!file->open(Core::OpenMode::ReadOnly)) { - warnln("Failed to open {}: {}", file->name(), file->error_string()); - return 1; - } + auto file = TRY(Core::File::open("/proc/df", Core::OpenMode::ReadOnly)); if (flag_human_readable) { outln("Filesystem Size Used Available Mount point"); @@ -47,7 +42,7 @@ int main(int argc, char** argv) } auto file_contents = file->read_all(); - auto json_result = JsonValue::from_string(file_contents).release_value_but_fixme_should_propagate_errors(); + auto json_result = TRY(JsonValue::from_string(file_contents)); auto const& json = json_result.as_array(); json.for_each([](auto& value) { auto& fs_object = value.as_object();