From 5caa9311d78fd9d79963bae86c9019a9169af5c9 Mon Sep 17 00:00:00 2001 From: Pedro Pereira Date: Thu, 25 Nov 2021 00:23:47 +0000 Subject: [PATCH] Breakout: TRY() all the things in serenity_main() :^) --- Userland/Games/Breakout/main.cpp | 21 +++++++++++---------- 1 file changed, 11 insertions(+), 10 deletions(-) diff --git a/Userland/Games/Breakout/main.cpp b/Userland/Games/Breakout/main.cpp index 4f4c9e7794..8cd361aa88 100644 --- a/Userland/Games/Breakout/main.cpp +++ b/Userland/Games/Breakout/main.cpp @@ -30,23 +30,24 @@ ErrorOr serenity_main(Main::Arguments arguments) window->set_resizable(false); window->set_double_buffering_enabled(false); window->set_title("Breakout"); + auto app_icon = GUI::Icon::default_icon("app-breakout"); window->set_icon(app_icon.bitmap_for_size(16)); - auto& game = window->set_main_widget(); - auto& game_menu = window->add_menu("&Game"); - game_menu.add_action(GUI::Action::create_checkable("&Pause", { {}, Key_P }, [&](auto& action) { - game.set_paused(action.is_checked()); - })); + auto game = TRY(window->try_set_main_widget()); - game_menu.add_separator(); + auto game_menu = TRY(window->try_add_menu("&Game")); + TRY(game_menu->try_add_action(GUI::Action::create_checkable("&Pause", { {}, Key_P }, [&](auto& action) { + game->set_paused(action.is_checked()); + }))); - game_menu.add_action(GUI::CommonActions::make_quit_action([](auto&) { + TRY(game_menu->try_add_separator()); + TRY(game_menu->try_add_action(GUI::CommonActions::make_quit_action([](auto&) { GUI::Application::the()->quit(); - })); + }))); - auto& help_menu = window->add_menu("&Help"); - help_menu.add_action(GUI::CommonActions::make_about_action("Breakout", app_icon, window)); + auto help_menu = TRY(window->try_add_menu("&Help")); + TRY(help_menu->try_add_action(GUI::CommonActions::make_about_action("Breakout", app_icon, window))); window->show();