From 542e18b367dc7d4a55e9abcda563207c182fdc7c Mon Sep 17 00:00:00 2001 From: Daste Date: Thu, 10 Feb 2022 17:42:53 +0100 Subject: [PATCH] HackStudio: Fix error handling logic in delete_action The `result.is_error()` check was inverted, causing a crash. --- Userland/DevTools/HackStudio/HackStudioWidget.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Userland/DevTools/HackStudio/HackStudioWidget.cpp b/Userland/DevTools/HackStudio/HackStudioWidget.cpp index ff1bf6de40..de9cc2a485 100644 --- a/Userland/DevTools/HackStudio/HackStudioWidget.cpp +++ b/Userland/DevTools/HackStudio/HackStudioWidget.cpp @@ -553,7 +553,7 @@ NonnullRefPtr HackStudioWidget::create_delete_action() } bool is_directory = S_ISDIR(st.st_mode); - if (auto result = Core::File::remove(file, Core::File::RecursionMode::Allowed, false); !result.is_error()) { + if (auto result = Core::File::remove(file, Core::File::RecursionMode::Allowed, false); result.is_error()) { auto& error = result.error(); if (is_directory) { GUI::MessageBox::show(window(),