Reviews log messages, fixed some typos and ensure consistency

This commit is contained in:
morg 2021-09-02 13:09:44 +02:00
parent 344e7c7551
commit 70024faabd
3 changed files with 10 additions and 10 deletions

View file

@ -54,11 +54,11 @@ void DetectPolychromeSMBusControllers(std::vector<i2c_smbus_interface*>& busses)
{ {
IF_MOBO_SMBUS(busses[bus]->pci_vendor, busses[bus]->pci_device) IF_MOBO_SMBUS(busses[bus]->pci_vendor, busses[bus]->pci_device)
{ {
LOG_TRACE("%s Bus %02d is an Motherboard, checking for for a device at 0x6A", ASROCK_DETECTOR_NAME, bus); LOG_TRACE("%s Bus %02d is a motherboard, checking for a device at 0x6A", ASROCK_DETECTOR_NAME, bus);
// Check for Polychrome controller at 0x6A // Check for Polychrome controller at 0x6A
if (TestForPolychromeSMBusController(busses[bus], 0x6A)) if (TestForPolychromeSMBusController(busses[bus], 0x6A))
{ {
LOG_TRACE("%s Detected a device at address 0x6A, Testing for a known controller", ASROCK_DETECTOR_NAME); LOG_TRACE("%s Detected a device at address 0x6A, testing for a known controller", ASROCK_DETECTOR_NAME);
new_polychrome = new PolychromeController(busses[bus], 0x6A); new_polychrome = new PolychromeController(busses[bus], 0x6A);
if(new_polychrome->GetASRockType() != ASROCK_TYPE_UNKNOWN) if(new_polychrome->GetASRockType() != ASROCK_TYPE_UNKNOWN)
@ -80,7 +80,7 @@ void DetectPolychromeSMBusControllers(std::vector<i2c_smbus_interface*>& busses)
} }
else else
{ {
LOG_TRACE("%s Bus %02d not a Motherboard", ASROCK_DETECTOR_NAME, bus); LOG_TRACE("%s Bus %02d not a motherboard", ASROCK_DETECTOR_NAME, bus);
} }
} }

View file

@ -9,7 +9,7 @@
#include "filesystem.h" #include "filesystem.h"
static const char* log_codes[] = {"FATAL:", "ERROR:", "Warning:", "Info:", "[verbose]", "Debug:", "Trace:"}; static const char* log_codes[] = {"FATAL:", "ERROR:", "Warning:", "Info:", "Verbose:", "Debug:", "Trace:"};
LogManager::LogManager() LogManager::LogManager()
{ {

View file

@ -105,7 +105,7 @@ std::vector<i2c_smbus_interface*> & ResourceManager::GetI2CBusses()
void ResourceManager::RegisterRGBController(RGBController *rgb_controller) void ResourceManager::RegisterRGBController(RGBController *rgb_controller)
{ {
LOG_INFO("Registering RGB controller: %s", rgb_controller->name.c_str()); LOG_INFO("[%s] Registering RGB controller", rgb_controller->name.c_str());
rgb_controllers_hw.push_back(rgb_controller); rgb_controllers_hw.push_back(rgb_controller);
UpdateDeviceList(); UpdateDeviceList();
@ -113,7 +113,7 @@ void ResourceManager::RegisterRGBController(RGBController *rgb_controller)
void ResourceManager::UnregisterRGBController(RGBController* rgb_controller) void ResourceManager::UnregisterRGBController(RGBController* rgb_controller)
{ {
LOG_INFO("Unregistering RGB controller: %s", rgb_controller->name.c_str()); LOG_INFO("[%s] Unregistering RGB controller", rgb_controller->name.c_str());
/*-------------------------------------------------------------------------*\ /*-------------------------------------------------------------------------*\
| Clear callbacks from the controller before removal | | Clear callbacks from the controller before removal |
@ -746,7 +746,7 @@ void ResourceManager::DetectDevicesThreadFunction()
this_device_enabled = detector_settings["detectors"][detection_string]; this_device_enabled = detector_settings["detectors"][detection_string];
} }
LOG_DEBUG("[%s] is %s", detection_string, ((this_device_enabled == true) ? "Enabled" : "Disabled")); LOG_DEBUG("[%s] is %s", detection_string, ((this_device_enabled == true) ? "enabled" : "disabled"));
if(this_device_enabled) if(this_device_enabled)
{ {
DetectionProgressChanged(); DetectionProgressChanged();
@ -845,7 +845,7 @@ void ResourceManager::DetectDevicesThreadFunction()
this_device_enabled = detector_settings["detectors"][detection_string]; this_device_enabled = detector_settings["detectors"][detection_string];
} }
LOG_DEBUG("[%s] is %s", detection_string, ((this_device_enabled == true) ? "Enabled" : "Disabled")); LOG_DEBUG("[%s] is %s", detection_string, ((this_device_enabled == true) ? "enabled" : "disabled"));
if(this_device_enabled) if(this_device_enabled)
{ {
@ -940,7 +940,7 @@ void ResourceManager::DetectDevicesThreadFunction()
this_device_enabled = detector_settings["detectors"][detection_string]; this_device_enabled = detector_settings["detectors"][detection_string];
} }
LOG_DEBUG("[%s] is %s", detection_string, ((this_device_enabled == true) ? "Enabled" : "Disabled")); LOG_DEBUG("[%s] is %s", detection_string, ((this_device_enabled == true) ? "enabled" : "disabled"));
if(this_device_enabled) if(this_device_enabled)
{ {
@ -1001,7 +1001,7 @@ void ResourceManager::DetectDevicesThreadFunction()
this_device_enabled = detector_settings["detectors"][detection_string]; this_device_enabled = detector_settings["detectors"][detection_string];
} }
LOG_DEBUG("[%s] is %s", detection_string, ((this_device_enabled == true) ? "Enabled" : "Disabled")); LOG_DEBUG("[%s] is %s", detection_string, ((this_device_enabled == true) ? "enabled" : "disabled"));
if(this_device_enabled) if(this_device_enabled)
{ {