From a93f6e51d355ee95009a07c8a574fc9eaed212bf Mon Sep 17 00:00:00 2001
From: Lioncash <mathew1800@gmail.com>
Date: Sun, 2 Aug 2020 20:17:42 -0400
Subject: [PATCH] emu_window: Mark Scoped constructor and Acquire() as
nodiscard
Ensures that callers make use of the constructor, preventing bugs from
silently occurring.
---
src/core/frontend/emu_window.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/core/frontend/emu_window.h b/src/core/frontend/emu_window.h
index 13aa149346..3e87802430 100644
--- a/src/core/frontend/emu_window.h
+++ b/src/core/frontend/emu_window.h
@@ -39,7 +39,7 @@ public:
class Scoped {
public:
- explicit Scoped(GraphicsContext& context_) : context(context_) {
+ [[nodiscard]] explicit Scoped(GraphicsContext& context_) : context(context_) {
context.MakeCurrent();
}
~Scoped() {
@@ -52,7 +52,7 @@ public:
/// Calls MakeCurrent on the context and calls DoneCurrent when the scope for the returned value
/// ends
- Scoped Acquire() {
+ [[nodiscard]] Scoped Acquire() {
return Scoped{*this};
}
};