From abf86b750100cc97bef0703c0e675bd2c30e4c2b Mon Sep 17 00:00:00 2001 From: pbrw Date: Thu, 25 Nov 2021 02:54:54 +0100 Subject: [PATCH] Fire: Use TRY() a lot more :^) --- Userland/Demos/Fire/CMakeLists.txt | 2 +- Userland/Demos/Fire/Fire.cpp | 39 ++++++++++++------------------ 2 files changed, 16 insertions(+), 25 deletions(-) diff --git a/Userland/Demos/Fire/CMakeLists.txt b/Userland/Demos/Fire/CMakeLists.txt index 31e7aaf90f..67480837c4 100644 --- a/Userland/Demos/Fire/CMakeLists.txt +++ b/Userland/Demos/Fire/CMakeLists.txt @@ -8,4 +8,4 @@ set(SOURCES ) serenity_app(Fire ICON app-fire) -target_link_libraries(Fire LibGUI LibCore LibGfx) +target_link_libraries(Fire LibGUI LibCore LibGfx LibMain) diff --git a/Userland/Demos/Fire/Fire.cpp b/Userland/Demos/Fire/Fire.cpp index 7e9fdbc155..63f98edd5c 100644 --- a/Userland/Demos/Fire/Fire.cpp +++ b/Userland/Demos/Fire/Fire.cpp @@ -23,6 +23,7 @@ */ #include +#include #include #include #include @@ -33,6 +34,7 @@ #include #include #include +#include #include #include #include @@ -195,40 +197,29 @@ void Fire::mouseup_event(GUI::MouseEvent& event) return GUI::Widget::mouseup_event(event); } -int main(int argc, char** argv) +ErrorOr serenity_main(Main::Arguments arguments) { - auto app = GUI::Application::construct(argc, argv); + auto app = TRY(GUI::Application::try_create(arguments)); - if (pledge("stdio recvfd sendfd rpath", nullptr) < 0) { - perror("pledge"); - return 1; - } + TRY(Core::System::pledge("stdio recvfd sendfd rpath", nullptr)); + TRY(Core::System::unveil("/res", "r")); + TRY(Core::System::unveil(nullptr, nullptr)); - if (unveil("/res", "r") < 0) { - perror("unveil"); - return 1; - } - - if (unveil(nullptr, nullptr) < 0) { - perror("unveil"); - return 1; - } - - auto window = GUI::Window::construct(); + auto window = TRY(GUI::Window::try_create()); window->set_double_buffering_enabled(false); window->set_title("Fire"); window->set_resizable(false); window->resize(FIRE_WIDTH * 2 + 4, FIRE_HEIGHT * 2 + 4); - auto& file_menu = window->add_menu("&File"); - file_menu.add_action(GUI::CommonActions::make_quit_action([&](auto&) { app->quit(); })); + auto file_menu = TRY(window->try_add_menu("&File")); + TRY(file_menu->try_add_action(GUI::CommonActions::make_quit_action([&](auto&) { app->quit(); }))); - auto& fire = window->set_main_widget(); + auto fire = TRY(window->try_set_main_widget()); - auto& time = fire.add(); - time.set_relative_rect({ 0, 4, 40, 10 }); - time.move_by({ window->width() - time.width(), 0 }); - fire.set_stat_label(time); + auto time = TRY(fire->try_add()); + time->set_relative_rect({ 0, 4, 40, 10 }); + time->move_by({ window->width() - time->width(), 0 }); + fire->set_stat_label(time); window->show();