Merge pull request #3112 from lioncash/skip

service/am: Remove unnecessary Skip calls
This commit is contained in:
bunnei 2019-11-21 15:30:01 -05:00 committed by GitHub
commit eedb048585
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -1336,12 +1336,16 @@ void IApplicationFunctions::GetPseudoDeviceId(Kernel::HLERequestContext& ctx) {
} }
void IApplicationFunctions::ExtendSaveData(Kernel::HLERequestContext& ctx) { void IApplicationFunctions::ExtendSaveData(Kernel::HLERequestContext& ctx) {
struct Parameters {
FileSys::SaveDataType type;
u128 user_id;
u64 new_normal_size;
u64 new_journal_size;
};
static_assert(sizeof(Parameters) == 40);
IPC::RequestParser rp{ctx}; IPC::RequestParser rp{ctx};
const auto type{rp.PopRaw<FileSys::SaveDataType>()}; const auto [type, user_id, new_normal_size, new_journal_size] = rp.PopRaw<Parameters>();
rp.Skip(1, false);
const auto user_id{rp.PopRaw<u128>()};
const auto new_normal_size{rp.PopRaw<u64>()};
const auto new_journal_size{rp.PopRaw<u64>()};
LOG_DEBUG(Service_AM, LOG_DEBUG(Service_AM,
"called with type={:02X}, user_id={:016X}{:016X}, new_normal={:016X}, " "called with type={:02X}, user_id={:016X}{:016X}, new_normal={:016X}, "
@ -1360,10 +1364,14 @@ void IApplicationFunctions::ExtendSaveData(Kernel::HLERequestContext& ctx) {
} }
void IApplicationFunctions::GetSaveDataSize(Kernel::HLERequestContext& ctx) { void IApplicationFunctions::GetSaveDataSize(Kernel::HLERequestContext& ctx) {
struct Parameters {
FileSys::SaveDataType type;
u128 user_id;
};
static_assert(sizeof(Parameters) == 24);
IPC::RequestParser rp{ctx}; IPC::RequestParser rp{ctx};
const auto type{rp.PopRaw<FileSys::SaveDataType>()}; const auto [type, user_id] = rp.PopRaw<Parameters>();
rp.Skip(1, false);
const auto user_id{rp.PopRaw<u128>()};
LOG_DEBUG(Service_AM, "called with type={:02X}, user_id={:016X}{:016X}", static_cast<u8>(type), LOG_DEBUG(Service_AM, "called with type={:02X}, user_id={:016X}{:016X}", static_cast<u8>(type),
user_id[1], user_id[0]); user_id[1], user_id[0]);