Skip to content

Commit 92dc841

Browse files
Fix Corsair mouse LED names by storing name in controller object
1 parent f0468ff commit 92dc841

4 files changed

Lines changed: 16 additions & 2 deletions

File tree

Controllers/CorsairPeripheralController/CorsairPeripheralController.cpp

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -91,6 +91,11 @@ std::string CorsairPeripheralController::GetFirmwareString()
9191
return firmware_version;
9292
}
9393

94+
std::string CorsairPeripheralController::GetName()
95+
{
96+
return name;
97+
}
98+
9499
void CorsairPeripheralController::SetLEDs(std::vector<RGBColor>colors)
95100
{
96101
switch(type)
@@ -270,6 +275,11 @@ void CorsairPeripheralController::SetLEDsKeyboardLimited(std::vector<RGBColor> c
270275
SubmitKeyboardLimitedColors(216);
271276
}
272277

278+
void CorsairPeripheralController::SetName(std::string device_name)
279+
{
280+
name = device_name;
281+
}
282+
273283
/*-------------------------------------------------------------------------------------------------*\
274284
| Private packet sending functions. |
275285
\*-------------------------------------------------------------------------------------------------*/

Controllers/CorsairPeripheralController/CorsairPeripheralController.h

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -74,18 +74,21 @@ class CorsairPeripheralController
7474
device_type GetDeviceType();
7575
std::string GetDeviceLocation();
7676
std::string GetFirmwareString();
77+
std::string GetName();
7778

7879
void SetLEDs(std::vector<RGBColor> colors);
7980
void SetLEDsKeyboardFull(std::vector<RGBColor> colors);
8081
void SetLEDsKeyboardLimited(std::vector<RGBColor> colors);
8182
void SetLEDsMouse(std::vector<RGBColor> colors);
8283
void SetLEDsMousemat(std::vector<RGBColor> colors);
84+
void SetName(std::string device_name);
8385

8486
private:
8587
hid_device* dev;
8688

8789
std::string firmware_version;
8890
std::string location;
91+
std::string name;
8992
device_type type;
9093
int physical_layout; //ANSI, ISO, etc.
9194
int logical_layout; //Normal, K95 or K95 Platinum

Controllers/CorsairPeripheralController/CorsairPeripheralControllerDetect.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,10 +73,11 @@ void DetectCorsairPeripheralControllers(hid_device_info* info, const std::string
7373
if( dev )
7474
{
7575
CorsairPeripheralController* controller = new CorsairPeripheralController(dev, info->path);
76+
controller->SetName(name);
77+
7678
if(controller->GetDeviceType() != DEVICE_TYPE_UNKNOWN)
7779
{
7880
RGBController_CorsairPeripheral* rgb_controller = new RGBController_CorsairPeripheral(controller);
79-
rgb_controller->name = name;
8081
ResourceManager::get()->RegisterRGBController(rgb_controller);
8182
}
8283
else

Controllers/CorsairPeripheralController/RGBController_CorsairPeripheral.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -574,7 +574,7 @@ RGBController_CorsairPeripheral::RGBController_CorsairPeripheral(CorsairPeripher
574574
{
575575
corsair = corsair_ptr;
576576

577-
name = "Corsair RGB Peripheral Device";
577+
name = corsair->GetName();
578578
vendor = "Corsair";
579579
description = "Corsair RGB Peripheral Device";
580580
type = corsair->GetDeviceType();

0 commit comments

Comments
 (0)