Properly scope pid inside to_decorations_type

Co-authored-by: Matthew Kosarek <[email protected]>
canonical · Jul 3, 2024 · fc2d435 · fc2d435
1 parent 2aeb440
commit fc2d435
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/src/server/frontend_wayland/xdg_decoration_unstable_v1.cpp b/src/server/frontend_wayland/xdg_decoration_unstable_v1.cpp
@@ -202,13 +202,15 @@ auto mir::frontend::XdgToplevelDecorationV1::to_decorations_type(uint32_t mode)
     case Mode::server_side:
         return DecorationStrategy::DecorationsType::ssd;
     default:
+    {
         pid_t pid;
         wl_client_get_credentials(client->raw_client(), &pid, nullptr, nullptr); // null pointers are allowed
 
         mir::log_warning("Client PID: %d, attempted to set invalid zxdg_toplevel_decoration_v1 mode (%d), defaulting to client side.", pid, mode);
 
         return DecorationStrategy::DecorationsType::csd;
     }
+    }
 }
 
 void mir::frontend::XdgToplevelDecorationV1::update_mode(uint32_t new_mode)