From dc41025ffc860594238ead0691cf2941c488924e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Vladim=C3=ADr=20Vondru=C5=A1?= Date: Sun, 21 Jun 2026 16:32:46 +0200 Subject: [PATCH] GL: remove stray unused expression. This originated in 7ad928ae4230b116128d142d980767a637c8bedc which looks like a generic find & replace that removed all early return statements because the unmasked renderer string on WebGL could expose both the underlying driver (NVidia, Mesa, ...) and the wrapper (ANGLE, ...). Ideally the compiler would warn about an unused expression, but Optional::operator*() isn't const on mutable variables, so it didn't. Only discovered this because I'm removing the Optional wrapper from this member, and then the compiler started to warn. --- src/Magnum/GL/Implementation/driverSpecific.cpp | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/Magnum/GL/Implementation/driverSpecific.cpp b/src/Magnum/GL/Implementation/driverSpecific.cpp index b5559c381..81e240058 100644 --- a/src/Magnum/GL/Implementation/driverSpecific.cpp +++ b/src/Magnum/GL/Implementation/driverSpecific.cpp @@ -591,8 +591,6 @@ auto Context::detectedDriver() -> DetectedDrivers { if(renderer.contains("SVGA3D"_s)) *_detectedDrivers |= DetectedDriver::Svga3D; - - *_detectedDrivers; } if(vendor.contains("NVIDIA Corporation"_s))