Unsound usage of Vec::from_raw_parts()

[shinmao]

Hi, we consider that the usage of Vec::from_raw_parts() here is unsound

Fyrox/fyrox-core/src/visitor.rs

Lines 554 to 559 in 4724d98

	*self.vec = unsafe {
	Vec::from_raw_parts(
	bytes.as_mut_ptr() as *mut T,
	bytes.len() / std::mem::size_of::<T>(),
	bytes.capacity() / std::mem::size_of::<T>(),
	)

First, since bytes is aligned to 1 byte, misaligned pointer will be passed to from_raw_parts, which is similar to the issue here: #538 (comment)

Second, since the code is trying to casting a pointer of Vec to the pointer of inner type where the Vec has unstable layout.

In the comments, we found the explanation that BinaryBlob has less type safety and it accepts any type T that is Copy, and it lacks the type_id system. Is this method designed for purpose?