beenull/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-09-30 00:31:14 +00:00
Code Issues Actions 5 Packages Projects Releases Wiki Activity

shot: Fix hanging due to zero sized Rect

Browse source 
shot was refactored to retrive crop_region,
but it is empty when -r is not used.
This commit is contained in:
Aziz Berkay Yesilyurt 2021-11-03 21:39:45 +01:00 committed by Andreas Kling
parent 0dae7c9420
commit 7f8794f902
Notes: sideshowbarker 2024-07-18 01:29:10 +09:00 
Author: https://github.com/abyesilyurt
Commit: https://github.com/SerenityOS/serenity/commit/7f8794f902a
Pull-request: https://github.com/SerenityOS/serenity/pull/10788
Reviewed-by: https://github.com/davidot ✅
Reviewed-by: https://github.com/kleinesfilmroellchen ✅
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
Userland/Utilities/shot.cpp
View file

@ -108,7 +108,7 @@ int main(int argc, char** argv)
}
auto app = GUI::Application::construct(argc, argv);
Gfx::IntRect crop_region;
Optional<Gfx::IntRect> crop_region;
if (select_region) {
auto window = GUI::Window::construct();
auto& container = window->set_main_widget<SelectableLayover>(window);
@ -120,7 +120,7 @@ int main(int argc, char** argv)
app->exec();
crop_region = container.region();
if (crop_region.is_empty()) {
if (crop_region.value().is_empty()) {
dbgln("cancelled...");
return 0;
}