diff options
| author | 2020-07-25 03:11:14 -0400 | |
|---|---|---|
| committer | 2020-07-25 03:11:56 -0400 | |
| commit | c5bdccfecb279ba581e5931f7fd9678d1aaa8f85 (patch) | |
| tree | 8d12aff0d6624729521ea9a5950308fd2731aa18 | |
| parent | Merge pull request #4383 from ogniK5377/dark-checkbox (diff) | |
| download | yuzu-c5bdccfecb279ba581e5931f7fd9678d1aaa8f85.tar.gz yuzu-c5bdccfecb279ba581e5931f7fd9678d1aaa8f85.tar.xz yuzu-c5bdccfecb279ba581e5931f7fd9678d1aaa8f85.zip | |
zstd_compression: Make use of std::span in interfaces
Allows condensing the data and size parameters into a single argument.
| -rw-r--r-- | src/common/zstd_compression.cpp | 13 | ||||
| -rw-r--r-- | src/common/zstd_compression.h | 7 | ||||
| -rw-r--r-- | src/video_core/renderer_opengl/gl_shader_disk_cache.cpp | 3 |
3 files changed, 10 insertions, 13 deletions
diff --git a/src/common/zstd_compression.cpp b/src/common/zstd_compression.cpp index 978526492..770833ee7 100644 --- a/src/common/zstd_compression.cpp +++ b/src/common/zstd_compression.cpp | |||
| @@ -5,19 +5,18 @@ | |||
| 5 | #include <algorithm> | 5 | #include <algorithm> |
| 6 | #include <zstd.h> | 6 | #include <zstd.h> |
| 7 | 7 | ||
| 8 | #include "common/assert.h" | ||
| 9 | #include "common/zstd_compression.h" | 8 | #include "common/zstd_compression.h" |
| 10 | 9 | ||
| 11 | namespace Common::Compression { | 10 | namespace Common::Compression { |
| 12 | 11 | ||
| 13 | std::vector<u8> CompressDataZSTD(const u8* source, std::size_t source_size, s32 compression_level) { | 12 | std::vector<u8> CompressDataZSTD(std::span<const u8> source, s32 compression_level) { |
| 14 | compression_level = std::clamp(compression_level, 1, ZSTD_maxCLevel()); | 13 | compression_level = std::clamp(compression_level, 1, ZSTD_maxCLevel()); |
| 15 | 14 | ||
| 16 | const std::size_t max_compressed_size = ZSTD_compressBound(source_size); | 15 | const std::size_t max_compressed_size = ZSTD_compressBound(source.size()); |
| 17 | std::vector<u8> compressed(max_compressed_size); | 16 | std::vector<u8> compressed(max_compressed_size); |
| 18 | 17 | ||
| 19 | const std::size_t compressed_size = | 18 | const std::size_t compressed_size = ZSTD_compress( |
| 20 | ZSTD_compress(compressed.data(), compressed.size(), source, source_size, compression_level); | 19 | compressed.data(), compressed.size(), source.data(), source.size(), compression_level); |
| 21 | 20 | ||
| 22 | if (ZSTD_isError(compressed_size)) { | 21 | if (ZSTD_isError(compressed_size)) { |
| 23 | // Compression failed | 22 | // Compression failed |
| @@ -29,8 +28,8 @@ std::vector<u8> CompressDataZSTD(const u8* source, std::size_t source_size, s32 | |||
| 29 | return compressed; | 28 | return compressed; |
| 30 | } | 29 | } |
| 31 | 30 | ||
| 32 | std::vector<u8> CompressDataZSTDDefault(const u8* source, std::size_t source_size) { | 31 | std::vector<u8> CompressDataZSTDDefault(std::span<const u8> source) { |
| 33 | return CompressDataZSTD(source, source_size, ZSTD_CLEVEL_DEFAULT); | 32 | return CompressDataZSTD(source, ZSTD_CLEVEL_DEFAULT); |
| 34 | } | 33 | } |
| 35 | 34 | ||
| 36 | std::vector<u8> DecompressDataZSTD(const std::vector<u8>& compressed) { | 35 | std::vector<u8> DecompressDataZSTD(const std::vector<u8>& compressed) { |
diff --git a/src/common/zstd_compression.h b/src/common/zstd_compression.h index e9de941c8..b5edf19e7 100644 --- a/src/common/zstd_compression.h +++ b/src/common/zstd_compression.h | |||
| @@ -4,6 +4,7 @@ | |||
| 4 | 4 | ||
| 5 | #pragma once | 5 | #pragma once |
| 6 | 6 | ||
| 7 | #include <span> | ||
| 7 | #include <vector> | 8 | #include <vector> |
| 8 | 9 | ||
| 9 | #include "common/common_types.h" | 10 | #include "common/common_types.h" |
| @@ -14,23 +15,21 @@ namespace Common::Compression { | |||
| 14 | * Compresses a source memory region with Zstandard and returns the compressed data in a vector. | 15 | * Compresses a source memory region with Zstandard and returns the compressed data in a vector. |
| 15 | * | 16 | * |
| 16 | * @param source the uncompressed source memory region. | 17 | * @param source the uncompressed source memory region. |
| 17 | * @param source_size the size in bytes of the uncompressed source memory region. | ||
| 18 | * @param compression_level the used compression level. Should be between 1 and 22. | 18 | * @param compression_level the used compression level. Should be between 1 and 22. |
| 19 | * | 19 | * |
| 20 | * @return the compressed data. | 20 | * @return the compressed data. |
| 21 | */ | 21 | */ |
| 22 | std::vector<u8> CompressDataZSTD(const u8* source, std::size_t source_size, s32 compression_level); | 22 | std::vector<u8> CompressDataZSTD(std::span<const u8> source, s32 compression_level); |
| 23 | 23 | ||
| 24 | /** | 24 | /** |
| 25 | * Compresses a source memory region with Zstandard with the default compression level and returns | 25 | * Compresses a source memory region with Zstandard with the default compression level and returns |
| 26 | * the compressed data in a vector. | 26 | * the compressed data in a vector. |
| 27 | * | 27 | * |
| 28 | * @param source the uncompressed source memory region. | 28 | * @param source the uncompressed source memory region. |
| 29 | * @param source_size the size in bytes of the uncompressed source memory region. | ||
| 30 | * | 29 | * |
| 31 | * @return the compressed data. | 30 | * @return the compressed data. |
| 32 | */ | 31 | */ |
| 33 | std::vector<u8> CompressDataZSTDDefault(const u8* source, std::size_t source_size); | 32 | std::vector<u8> CompressDataZSTDDefault(std::span<const u8> source); |
| 34 | 33 | ||
| 35 | /** | 34 | /** |
| 36 | * Decompresses a source memory region with Zstandard and returns the uncompressed data in a vector. | 35 | * Decompresses a source memory region with Zstandard and returns the uncompressed data in a vector. |
diff --git a/src/video_core/renderer_opengl/gl_shader_disk_cache.cpp b/src/video_core/renderer_opengl/gl_shader_disk_cache.cpp index 2dcc2b0eb..c0e73789b 100644 --- a/src/video_core/renderer_opengl/gl_shader_disk_cache.cpp +++ b/src/video_core/renderer_opengl/gl_shader_disk_cache.cpp | |||
| @@ -422,8 +422,7 @@ void ShaderDiskCacheOpenGL::SavePrecompiledHeaderToVirtualPrecompiledCache() { | |||
| 422 | void ShaderDiskCacheOpenGL::SaveVirtualPrecompiledFile() { | 422 | void ShaderDiskCacheOpenGL::SaveVirtualPrecompiledFile() { |
| 423 | precompiled_cache_virtual_file_offset = 0; | 423 | precompiled_cache_virtual_file_offset = 0; |
| 424 | const std::vector<u8> uncompressed = precompiled_cache_virtual_file.ReadAllBytes(); | 424 | const std::vector<u8> uncompressed = precompiled_cache_virtual_file.ReadAllBytes(); |
| 425 | const std::vector<u8> compressed = | 425 | const std::vector<u8> compressed = Common::Compression::CompressDataZSTDDefault(uncompressed); |
| 426 | Common::Compression::CompressDataZSTDDefault(uncompressed.data(), uncompressed.size()); | ||
| 427 | 426 | ||
| 428 | const auto precompiled_path{GetPrecompiledPath()}; | 427 | const auto precompiled_path{GetPrecompiledPath()}; |
| 429 | FileUtil::IOFile file(precompiled_path, "wb"); | 428 | FileUtil::IOFile file(precompiled_path, "wb"); |