diff options
| author | jduncanator <1518948+jduncanator@users.noreply.github.com> | 2019-01-11 11:11:46 +1100 |
|---|---|---|
| committer | Ac_K <Acoustik666@gmail.com> | 2019-01-11 01:11:46 +0100 |
| commit | 8406ec6272392a3f7f7672d85fdde333b6c70378 (patch) | |
| tree | b8d3f0a1692015c4bd56897cf7146877aa967da0 /Ryujinx.HLE/HOS/Services/Psm | |
| parent | 600799ba87fd59c3a6ef56e55a689a1ebdc410a6 (diff) | |
Refactor Ryujinx.Common and HLE Stub Logging (#537)
* Refactor Ryujinx.Common and HLE Stub Logging
* Resolve review comments
* Rename missed loop variable
* Optimize PrintStub logging function
* Pass the call-sites Thread ID through to the logger
* Remove superfluous lock from ConsoleLog
* Process logged data objects in the logger target
Pass the data object all the way to the output logger targets, to allow them to "serialize" this in whatever appropriate format they're logging in.
* Use existing StringBuilder to build the properties string
* Add a ServiceNotImplemented Exception
Useful for printing debug information about unimplemented service calls
* Resolve Style Nits
* Resolve Merge Issues
* Fix typo and align declarations
Diffstat (limited to 'Ryujinx.HLE/HOS/Services/Psm')
| -rw-r--r-- | Ryujinx.HLE/HOS/Services/Psm/IPsmServer.cs | 8 | ||||
| -rw-r--r-- | Ryujinx.HLE/HOS/Services/Psm/IPsmSession.cs | 10 |
2 files changed, 10 insertions, 8 deletions
diff --git a/Ryujinx.HLE/HOS/Services/Psm/IPsmServer.cs b/Ryujinx.HLE/HOS/Services/Psm/IPsmServer.cs index c7f6083c..02bc5524 100644 --- a/Ryujinx.HLE/HOS/Services/Psm/IPsmServer.cs +++ b/Ryujinx.HLE/HOS/Services/Psm/IPsmServer.cs @@ -34,7 +34,7 @@ namespace Ryujinx.HLE.HOS.Services.Psm context.ResponseData.Write(chargePercentage); - Logger.PrintStub(LogClass.ServicePsm, $"Stubbed. ChargePercentage: {chargePercentage}"); + Logger.PrintStub(LogClass.ServicePsm, new { chargePercentage }); return 0; } @@ -42,9 +42,11 @@ namespace Ryujinx.HLE.HOS.Services.Psm // GetChargerType() -> u32 public static long GetChargerType(ServiceCtx context) { - context.ResponseData.Write((int)ChargerType.ChargerOrDock); + ChargerType chargerType = ChargerType.ChargerOrDock; - Logger.PrintStub(LogClass.ServicePsm, $"Stubbed. ChargerType: {ChargerType.ChargerOrDock}"); + context.ResponseData.Write((int)chargerType); + + Logger.PrintStub(LogClass.ServicePsm, new { chargerType }); return 0; } diff --git a/Ryujinx.HLE/HOS/Services/Psm/IPsmSession.cs b/Ryujinx.HLE/HOS/Services/Psm/IPsmSession.cs index 0c304b41..aeeeb052 100644 --- a/Ryujinx.HLE/HOS/Services/Psm/IPsmSession.cs +++ b/Ryujinx.HLE/HOS/Services/Psm/IPsmSession.cs @@ -45,7 +45,7 @@ namespace Ryujinx.HLE.HOS.Services.Psm context.Response.HandleDesc = IpcHandleDesc.MakeCopy(_stateChangeEventHandle); - Logger.PrintStub(LogClass.ServicePsm, "Stubbed."); + Logger.PrintStub(LogClass.ServicePsm); return 0; } @@ -59,7 +59,7 @@ namespace Ryujinx.HLE.HOS.Services.Psm _stateChangeEventHandle = -1; } - Logger.PrintStub(LogClass.ServicePsm, "Stubbed."); + Logger.PrintStub(LogClass.ServicePsm); return 0; } @@ -69,7 +69,7 @@ namespace Ryujinx.HLE.HOS.Services.Psm { bool chargerTypeChangeEventEnabled = context.RequestData.ReadBoolean(); - Logger.PrintStub(LogClass.ServicePsm, $"Stubbed. ChargerTypeChangeEventEnabled: {chargerTypeChangeEventEnabled}"); + Logger.PrintStub(LogClass.ServicePsm, new { chargerTypeChangeEventEnabled }); return 0; } @@ -79,7 +79,7 @@ namespace Ryujinx.HLE.HOS.Services.Psm { bool powerSupplyChangeEventEnabled = context.RequestData.ReadBoolean(); - Logger.PrintStub(LogClass.ServicePsm, $"Stubbed. PowerSupplyChangeEventEnabled: {powerSupplyChangeEventEnabled}"); + Logger.PrintStub(LogClass.ServicePsm, new { powerSupplyChangeEventEnabled }); return 0; } @@ -89,7 +89,7 @@ namespace Ryujinx.HLE.HOS.Services.Psm { bool batteryVoltageStateChangeEventEnabled = context.RequestData.ReadBoolean(); - Logger.PrintStub(LogClass.ServicePsm, $"Stubbed. BatteryVoltageStateChangeEventEnabled: {batteryVoltageStateChangeEventEnabled}"); + Logger.PrintStub(LogClass.ServicePsm, new { batteryVoltageStateChangeEventEnabled }); return 0; } |