dburr/eden
1
0
Fork 0
mirror of https://git.eden-emu.dev/eden-emu/eden.git synced 2025-07-23 15:35:46 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

fsp_srv: Remove unnecessary vector construction in IFile's Write() function

Browse source 
We can avoid constructing a std::vector here by simply passing a pointer
to the original data and the size of the copy we wish to perform to the
backend's Write() function instead, avoiding copying the data where it's
otherwise not needed.
This commit is contained in:
Lioncash 2018-07-19 11:01:05 -04:00
parent 45e747beaa
commit 6fff93da5f
1 changed files with 3 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

5
src/core/hle/service/filesystem/fsp_srv.cpp
View file

@ -149,8 +149,9 @@ private:
length, data.size()); length, data.size());
// Write the data to the Storage backend // Write the data to the Storage backend
std::vector<u8> actual_data(data.begin(), data.begin() + length); const auto write_size =
const std::size_t written = backend->WriteBytes(std::move(actual_data), offset); static_cast<std::size_t>(std::distance(data.begin(), data.begin() + length));
const std::size_t written = backend->Write(data.data(), write_size, offset);
ASSERT_MSG(static_cast<s64>(written) == length, ASSERT_MSG(static_cast<s64>(written) == length,
"Could not write all bytes to file (requested={:016X}, actual={:016X}).", length, "Could not write all bytes to file (requested={:016X}, actual={:016X}).", length,