Skip to content

Commit

Permalink
don't export the CRC32 table
Browse files Browse the repository at this point in the history
It was not exported properly when building a DLL. We it's not necessary
for other classes.
It can be restrictued to that C++ compilation unit that noone else will see.
  • Loading branch information
robUx4 committed Dec 28, 2023
1 parent 83a710b commit b4b56a6
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
1 change: 0 additions & 1 deletion ebml/EbmlCrc32.h
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,6 @@ DECLARE_EBML_BINARY(EbmlCrc32)
void ResetCRC();
void UpdateByte(binary b);

static const std::array<std::uint32_t, 256> m_tab;
std::uint32_t m_crc;
std::uint32_t m_crc_final{0};

Expand Down
2 changes: 1 addition & 1 deletion src/EbmlCrc32.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ namespace libebml {

DEFINE_EBML_CLASS_GLOBAL(EbmlCrc32, 0xBF, 1, "EBMLCrc32\0ratamadabapa")

constexpr std::array<std::uint32_t, 256> EbmlCrc32::m_tab {
static constexpr std::array<std::uint32_t, 256> m_tab {
#ifdef WORDS_BIGENDIAN
0x00000000L, 0x96300777L, 0x2c610eeeL, 0xba510999L, 0x19c46d07L,
0x8ff46a70L, 0x35a563e9L, 0xa395649eL, 0x3288db0eL, 0xa4b8dc79L,
Expand Down

0 comments on commit b4b56a6

Please sign in to comment.