third_party/exiv2
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

replace array with vector

Browse Source 
Using the constructor is one less line. Plus it seems to fix bugs in
Apple's clang.

Signed-off-by: Rosen Penev <rosenp@gmail.com>
This commit is contained in:
Rosen Penev 2023-02-11 17:44:33 -08:00
parent 63b32ba3fb
commit 2d1691c1e2
1 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/preview.cpp
View File

@ -823,8 +823,7 @@ bool LoaderXmpJpeg::readDimensions() {
DataBuf decodeHex(const byte* src, size_t srcSize) {
// create decoding table
byte invalid = 16;
std::array<byte, 256> decodeHexTable;
decodeHexTable.fill(invalid);
auto decodeHexTable = std::vector<byte>(256, invalid);
for (byte i = 0; i < 10; i++)
decodeHexTable[static_cast<byte>('0') + i] = i;
for (byte i = 0; i < 6; i++)
@ -865,8 +864,7 @@ DataBuf decodeBase64(const std::string& src) {
// create decoding table
unsigned long invalid = 64;
std::array<unsigned long, 256> decodeBase64Table;
decodeBase64Table.fill(invalid);
auto decodeBase64Table = std::vector<unsigned long>(256, invalid);
for (unsigned long i = 0; i < 64; i++)
decodeBase64Table[static_cast<unsigned char>(encodeBase64Table[i])] = i;