Force `EventHandlerResult` values to be a single byte (#1131)

By specifying the type of the `EventHandlerResult` enum as `uint8_t`, it only
takes up one byte instead of two, saving a significant amout of PROGMEM.

Signed-off-by: Michael Richters <gedankenexperimenter@gmail.com>
pull/1132/head
Michael Richters 3 years ago committed by GitHub
parent 3d3b531571
commit 5a9025123e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

@ -1,5 +1,5 @@
/* Kaleidoscope - Firmware for computer input devices /* Kaleidoscope - Firmware for computer input devices
* Copyright (C) 2013-2018 Keyboard.io, Inc. * Copyright (C) 2013-2022 Keyboard.io, Inc.
* *
* This program is free software: you can redistribute it and/or modify it under * This program is free software: you can redistribute it and/or modify it under
* the terms of the GNU General Public License as published by the Free Software * the terms of the GNU General Public License as published by the Free Software
@ -16,6 +16,8 @@
#pragma once #pragma once
#include <stdint.h>
namespace kaleidoscope { namespace kaleidoscope {
// This is the set of return values for event handlers. Event handlers for // This is the set of return values for event handlers. Event handlers for
@ -41,7 +43,7 @@ namespace kaleidoscope {
// further handlers. There is currently no specification for what should // further handlers. There is currently no specification for what should
// happen if this is returned. // happen if this is returned.
enum class EventHandlerResult { enum class EventHandlerResult : uint8_t {
OK, OK,
EVENT_CONSUMED, EVENT_CONSUMED,
ABORT, ABORT,

Loading…
Cancel
Save