Merge pull request Tencent#1988 from Tencent/revert-1987-fix-placement-new-alignment

miloyip · web-flow · commit e4bde977440d · 2022-02-09T10:48:22.000+08:00
Revert "Fix the alignment of placement new buffer for GenericValue."
diff --git a/include/rapidjson/document.h b/include/rapidjson/document.h
@@ -1235,8 +1235,8 @@ class GenericValue {
             // return NullValue;
 
             // Use static buffer and placement-new to prevent destruction
-            static GenericValid buffer;
-            return *new (reinterpret_cast<char *>(&buffer)) GenericValue();
+            static char buffer[sizeof(GenericValue)];
+            return *new (buffer) GenericValue();
         }
     }
     template <typename SourceAllocator>

Original file line number	Diff line number	Diff line change
`@@ -1235,8 +1235,8 @@ class GenericValue {`
`1235`	`1235`	`// return NullValue;`
`1236`	`1236`
`1237`	`1237`	`// Use static buffer and placement-new to prevent destruction`
`1238`		`- static GenericValid buffer;`
`1239`		`- return new (reinterpret_cast<char >(&buffer)) GenericValue();`
	`1238`	`+ static char buffer[sizeof(GenericValue)];`
	`1239`	`+ return *new (buffer) GenericValue();`
`1240`	`1240`	`}`
`1241`	`1241`	`}`
`1242`	`1242`	`template <typename SourceAllocator>`