From 39de6fc1751c777c1acc40c6dbbb3558ed87055f Mon Sep 17 00:00:00 2001 From: Adam Honse Date: Tue, 18 May 2021 22:56:03 -0500 Subject: [PATCH] Code style cleanup for Logitech G Lightsync RGBController --- .../RGBController_LogitechGLightsync.cpp | 15 ++++++++++----- .../RGBController_LogitechGLightsync1zone.cpp | 15 ++++++++++----- .../RGBController_LogitechGPowerPlay.cpp | 15 ++++++++++----- 3 files changed, 30 insertions(+), 15 deletions(-) diff --git a/Controllers/LogitechController/RGBController_LogitechGLightsync.cpp b/Controllers/LogitechController/RGBController_LogitechGLightsync.cpp index 49045e831..0f1d4300a 100644 --- a/Controllers/LogitechController/RGBController_LogitechGLightsync.cpp +++ b/Controllers/LogitechController/RGBController_LogitechGLightsync.cpp @@ -11,7 +11,7 @@ RGBController_LogitechGLightsync::RGBController_LogitechGLightsync(LogitechGLightsyncController* logitech_ptr) { - logitech = logitech_ptr; + logitech = logitech_ptr; name = "Logitech G Lightsync Mouse"; vendor = "Logitech"; @@ -119,7 +119,9 @@ void RGBController_LogitechGLightsync::UpdateZoneLEDs(int zone) unsigned char grn = RGBGetGValue(colors[zone]); unsigned char blu = RGBGetBValue(colors[zone]); - // Replace direct mode with static when sending to controller. + /*---------------------------------------------------------*\ + | Replace direct mode with static when sending to controller| + \*---------------------------------------------------------*/ unsigned char temp_mode = (modes[active_mode].value != 0xFF) ? modes[active_mode].value : LOGITECH_G_LIGHTSYNC_MODE_STATIC; logitech->UpdateMouseLED(temp_mode, modes[active_mode].speed, zone, red, grn, blu, /* Brightness */ 0x64); @@ -132,13 +134,16 @@ void RGBController_LogitechGLightsync::UpdateSingleLED(int led) void RGBController_LogitechGLightsync::SetCustomMode() { - active_mode = 1; //Hard coded to first value in list + active_mode = 1; } void RGBController_LogitechGLightsync::DeviceUpdateMode() { - // If direct mode is true, then sent the packet to put the mouse in direct mode - // This code will only be called when we change modes as to not spam the device. + /*---------------------------------------------------------*\ + | If direct mode is true, then sent the packet to put the | + | mouse in direct mode. This code will only be called when | + | we change modes as to not spam the device. | + \*---------------------------------------------------------*/ logitech->SetDirectMode(modes[active_mode].value == 0xFF); DeviceUpdateLEDs(); } \ No newline at end of file diff --git a/Controllers/LogitechController/RGBController_LogitechGLightsync1zone.cpp b/Controllers/LogitechController/RGBController_LogitechGLightsync1zone.cpp index 39a60eae9..548971b1b 100644 --- a/Controllers/LogitechController/RGBController_LogitechGLightsync1zone.cpp +++ b/Controllers/LogitechController/RGBController_LogitechGLightsync1zone.cpp @@ -11,7 +11,7 @@ RGBController_LogitechGLightsync1zone::RGBController_LogitechGLightsync1zone(LogitechGLightsyncController* logitech_ptr) { - logitech = logitech_ptr; + logitech = logitech_ptr; name = "Logitech G Lightsync Mouse Single Zone"; vendor = "Logitech"; @@ -105,7 +105,9 @@ void RGBController_LogitechGLightsync1zone::UpdateZoneLEDs(int zone) unsigned char grn = RGBGetGValue(colors[zone]); unsigned char blu = RGBGetBValue(colors[zone]); - // Replace direct mode with static when sending to controller. + /*---------------------------------------------------------*\ + | Replace direct mode with static when sending to controller| + \*---------------------------------------------------------*/ unsigned char temp_mode = (modes[active_mode].value != 0xFF) ? modes[active_mode].value : LOGITECH_G_LIGHTSYNC_MODE_STATIC; logitech->UpdateMouseLED(temp_mode, modes[active_mode].speed, zone, red, grn, blu, /* Brightness */ 0x64); @@ -118,13 +120,16 @@ void RGBController_LogitechGLightsync1zone::UpdateSingleLED(int led) void RGBController_LogitechGLightsync1zone::SetCustomMode() { - active_mode = 1; //Hard coded to first value in list + active_mode = 1; } void RGBController_LogitechGLightsync1zone::DeviceUpdateMode() { - // If direct mode is true, then sent the packet to put the mouse in direct mode - // This code will only be called when we change modes as to not spam the device. + /*---------------------------------------------------------*\ + | If direct mode is true, then sent the packet to put the | + | mouse in direct mode. This code will only be called when | + | we change modes as to not spam the device. | + \*---------------------------------------------------------*/ logitech->SetDirectMode(modes[active_mode].value == 0xFF); DeviceUpdateLEDs(); } \ No newline at end of file diff --git a/Controllers/LogitechController/RGBController_LogitechGPowerPlay.cpp b/Controllers/LogitechController/RGBController_LogitechGPowerPlay.cpp index a6e23dda0..e3ec1c889 100644 --- a/Controllers/LogitechController/RGBController_LogitechGPowerPlay.cpp +++ b/Controllers/LogitechController/RGBController_LogitechGPowerPlay.cpp @@ -11,7 +11,7 @@ RGBController_LogitechGPowerPlay::RGBController_LogitechGPowerPlay(LogitechGLightsyncController* logitech_ptr) { - logitech = logitech_ptr; + logitech = logitech_ptr; name = "Logitech G PowerPlay Wireless Charging System"; vendor = "Logitech"; @@ -106,7 +106,9 @@ void RGBController_LogitechGPowerPlay::UpdateZoneLEDs(int zone) unsigned char grn = RGBGetGValue(colors[zone]); unsigned char blu = RGBGetBValue(colors[zone]); - // Replace direct mode with static when sending to controller. + /*---------------------------------------------------------*\ + | Replace direct mode with static when sending to controller| + \*---------------------------------------------------------*/ unsigned char temp_mode = (modes[active_mode].value != 0xFF) ? modes[active_mode].value : LOGITECH_G_LIGHTSYNC_MODE_STATIC; logitech->UpdateMouseLED(temp_mode, modes[active_mode].speed, zone, red, grn, blu, /* Brightness */ 0x64); @@ -119,13 +121,16 @@ void RGBController_LogitechGPowerPlay::UpdateSingleLED(int led) void RGBController_LogitechGPowerPlay::SetCustomMode() { - active_mode = 1; //Hard coded to first value in list + active_mode = 1; } void RGBController_LogitechGPowerPlay::DeviceUpdateMode() { - // If direct mode is true, then sent the packet to put the mouse in direct mode - // This code will only be called when we change modes as to not spam the device. + /*---------------------------------------------------------*\ + | If direct mode is true, then sent the packet to put the | + | mouse in direct mode. This code will only be called when | + | we change modes as to not spam the device. | + \*---------------------------------------------------------*/ logitech->SetDirectMode(modes[active_mode].value == 0xFF); DeviceUpdateLEDs(); } \ No newline at end of file