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, 4 insertions, 4 deletions

diff --git a/Minecraft.Client/RemotePlayer.cpp b/Minecraft.Client/RemotePlayer.cpp
index bea12842..dcfd6992 100644
--- a/Minecraft.Client/RemotePlayer.cpp
+++ b/Minecraft.Client/RemotePlayer.cpp
@@ -58,7 +58,7 @@ void RemotePlayer::tick()
     walkAnimSpeed += (wst - walkAnimSpeed) * 0.4f;
     walkAnimPos += walkAnimSpeed;
 
-	if (!hasStartedUsingItem && isUsingItemFlag() && inventory->items[inventory->selected] != NULL)
+	if (!hasStartedUsingItem && isUsingItemFlag() && inventory->items[inventory->selected] != nullptr)
 	{
 		shared_ptr<ItemInstance> item = inventory->items[inventory->selected];
 		startUsingItem(inventory->items[inventory->selected], Item::items[item->id]->getUseDuration(item));
@@ -103,8 +103,8 @@ void RemotePlayer::aiStep()
         while (yrd >= 180)
             yrd -= 360;
 
-        yRot += (float)((yrd) / lSteps);
-        xRot += (float)((lxr - xRot) / lSteps);
+        yRot += static_cast<float>((yrd) / lSteps);
+        xRot += static_cast<float>((lxr - xRot) / lSteps);
 
         lSteps--;
         setPos(xt, yt, zt);
@@ -113,7 +113,7 @@ void RemotePlayer::aiStep()
     oBob = bob;
 
     float tBob = (float) Mth::sqrt(xd * xd + zd * zd);
-    float tTilt = (float) atan(-yd * 0.2f) * 15.0f;
+    float tTilt = static_cast<float>(atan(-yd * 0.2f)) * 15.0f;
     if (tBob > 0.1f) tBob = 0.1f;
     if (!onGround || getHealth() <= 0) tBob = 0;
     if (onGround || getHealth() <= 0) tTilt = 0;