Mark getter functions as const (properly)

This commit is contained in:
oscgonfer
2026-06-14 14:48:40 +02:00
parent 2ef8d9cd3a
commit fc5dc5fc90
+6 -6
View File
@@ -73,9 +73,9 @@ class Screen
void showOverlayBanner(BannerOverlayOptions) {} void showOverlayBanner(BannerOverlayOptions) {}
void setFrames(FrameFocus focus) {} void setFrames(FrameFocus focus) {}
void endAlert() {} void endAlert() {}
const bool getIsI2cScreen() { return false; } bool getIsI2cScreen() const { return false; }
const uint32_t getI2cFrequency() { return 0; } uint32_t getI2cFrequency() const { return 0; }
ScanI2C::I2CPort getI2CPort() { return ScanI2C::I2CPort::NO_I2C; } ScanI2C::I2CPort getI2CPort() const { return ScanI2C::I2CPort::NO_I2C; }
}; };
} // namespace graphics } // namespace graphics
#else #else
@@ -263,16 +263,16 @@ class Screen : public concurrency::OSThread
Screen &operator=(const Screen &) = delete; Screen &operator=(const Screen &) = delete;
ScanI2C::DeviceAddress address_found; ScanI2C::DeviceAddress address_found;
const bool getIsI2cScreen() { return isI2cScreen; } bool getIsI2cScreen() const { return isI2cScreen; }
// Return I2C Speed, or 0 if none // Return I2C Speed, or 0 if none
const uint32_t getI2cFrequency() uint32_t getI2cFrequency() const
{ {
if (getIsI2cScreen()) if (getIsI2cScreen())
return dispdev->getI2cFrequency(); return dispdev->getI2cFrequency();
else else
return 0; return 0;
} }
const ScanI2C::I2CPort getI2CPort() ScanI2C::I2CPort getI2CPort() const
{ {
if (getIsI2cScreen()) if (getIsI2cScreen())
return address_found.port; return address_found.port;