Modernize project codebase (#906)

* Fixed boats falling and a TP glitch #266

* Replaced every C-style cast with C++ ones

* Replaced every C-style cast with C++ ones

* Fixed boats falling and a TP glitch #266

* Updated NULL to nullptr and fixing some type issues

* Modernized and fixed a few bugs

- Replaced most instances of `NULL` with `nullptr`.
- Replaced most `shared_ptr(new ...)` with `make_shared`.
- Removed the `nullptr` macro as it was interfering with the actual nullptr keyword in some instances.

* Fixing more conflicts

* Replace int loops with size_t and start work on overrides

* Add safety checks and fix a issue with vector going OOR

1 files changed, 6 insertions, 6 deletions

diff --git a/Minecraft.World/SetEntityMotionPacket.cpp b/Minecraft.World/SetEntityMotionPacket.cpp
index a0749b74..20d841f1 100644
--- a/Minecraft.World/SetEntityMotionPacket.cpp
+++ b/Minecraft.World/SetEntityMotionPacket.cpp
@@ -17,9 +17,9 @@ void SetEntityMotionPacket::_init(int id, double xd, double yd, double zd)
 	if (xd > m) xd = m;
 	if (yd > m) yd = m;
 	if (zd > m) zd = m;
-	xa = (int) (xd * 8000.0);
-	ya = (int) (yd * 8000.0);
-	za = (int) (zd * 8000.0);
+	xa = static_cast<int>(xd * 8000.0);
+	ya = static_cast<int>(yd * 8000.0);
+	za = static_cast<int>(zd * 8000.0);
 	// 4J - if we could transmit this as bytes (in 1/16 accuracy) then flag to do so
 	if( ( xa >= (-128 * 16 ) ) && ( ya >= (-128 * 16 ) ) && ( za >= (-128 * 16 ) ) &&
 		( xa < (128 * 16 ) ) && ( ya < (128 * 16 ) ) && ( za < (128 * 16 ) ) )
@@ -53,9 +53,9 @@ void SetEntityMotionPacket::read(DataInputStream *dis) //throws IOException
 	id = idAndFlag & 0x07ff;
 	if( idAndFlag & 0x0800 )
 	{
-		xa = (int)dis->readByte();
-		ya = (int)dis->readByte();
-		za = (int)dis->readByte();
+		xa = static_cast<int>(dis->readByte());
+		ya = static_cast<int>(dis->readByte());
+		za = static_cast<int>(dis->readByte());
 		xa = ( xa << 24 ) >> 24;
 		ya = ( ya << 24 ) >> 24;
 		za = ( za << 24 ) >> 24;