generic-library/poco
1
0
Fork 0
mirror of https://github.com/pocoproject/poco.git synced 2025-02-23 07:12:45 +01:00
Code Issues Projects Releases Wiki Activity

Remove std::aligned_storage as it is deprecated in C++23.

Browse Source 
Instead, replace it with std::max_align_t in the Any implementation, as
really we would like to be able to store any object with any alignment
in the small object optimization case. Typically the size and alignment
of std::max_align_t is 8 or 16 on most platforms. Added a static assert
to ensure that this change doesn't result in wasting more unused memory
in case the size of the storage buffer is smaller than this maximum
alignment (which is right now 64, so shall be ok on all platforms).
This commit is contained in:
Romain Geissler 2023-07-18 07:05:32 +00:00
parent bce5f961dd
commit d28129cbc4
No known key found for this signature in database
GPG Key ID: 2530E79BC9A4BB13
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
Foundation/include/Poco/Any.h
View File

@ -21,6 +21,7 @@
#include <algorithm>
#include <typeinfo>
#include <cstring>
#include <cstddef>
#define poco_any_assert(cond) do { if (!(cond)) std::abort(); } while (0)
@ -137,7 +138,8 @@ public:
}
private:
typedef typename std::aligned_storage<SizeV+1>::type AlignerType;
typedef std::max_align_t AlignerType;
static_assert(sizeof(AlignerType) >= SizeV + 1, "Aligner type is bigger than the actual storage, so SizeV should be made bigger otherwise you simply waste unused memory.");
void setLocal(bool local) const
{