From c22ac3367e86aba94914ce9fc2590c1e817c3009 Mon Sep 17 00:00:00 2001 From: David Griswold Date: Sat, 8 Mar 2025 14:47:39 -0300 Subject: [PATCH] rearrange switch cases to satisfy linux compiler --- src/core/frontend/framebuffer_layout.cpp | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/core/frontend/framebuffer_layout.cpp b/src/core/frontend/framebuffer_layout.cpp index 133655a11..df172b587 100644 --- a/src/core/frontend/framebuffer_layout.cpp +++ b/src/core/frontend/framebuffer_layout.cpp @@ -300,14 +300,16 @@ FramebufferLayout SeparateWindowsLayout(u32 width, u32 height, bool is_secondary FramebufferLayout AndroidSecondaryLayout(u32 width, u32 height) { const Settings::SecondaryScreenLayout layout = Settings::values.secondary_screen_layout.GetValue(); switch (layout) { - case Settings::SecondaryScreenLayout::None: - // this should never happen, but if it does, somehow, send the top screen - case Settings::SecondaryScreenLayout::TopScreenOnly: - return SingleFrameLayout(width, height, false, Settings::values.upright_screen.GetValue()); + case Settings::SecondaryScreenLayout::BottomScreenOnly: return SingleFrameLayout(width, height, true, Settings::values.upright_screen.GetValue()); case Settings::SecondaryScreenLayout::SideBySide: return LargeFrameLayout(width,height,false,Settings::values.upright_screen.GetValue(),1.0f,Settings::SmallScreenPosition::MiddleRight); + case Settings::SecondaryScreenLayout::None: + // this should never happen, but if it does, somehow, send the top screen + case Settings::SecondaryScreenLayout::TopScreenOnly: + default: + return SingleFrameLayout(width, height, false, Settings::values.upright_screen.GetValue()); } }