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

diff --git a/Minecraft.Client/ChestRenderer.cpp b/Minecraft.Client/ChestRenderer.cpp
index 97954b55..7d306f45 100644
--- a/Minecraft.Client/ChestRenderer.cpp
+++ b/Minecraft.Client/ChestRenderer.cpp
@@ -52,19 +52,19 @@ void ChestRenderer::render(shared_ptr<TileEntity>  _chest, double x, double y, d
 		Tile *tile = chest->getTile();
 		data = chest->getData();
 
-		if (dynamic_cast<ChestTile*>(tile) != NULL && data == 0)
+		if (dynamic_cast<ChestTile*>(tile) != nullptr && data == 0)
 		{
-			((ChestTile *) tile)->recalcLockDir(chest->getLevel(), chest->x, chest->y, chest->z);
+			static_cast<ChestTile *>(tile)->recalcLockDir(chest->getLevel(), chest->x, chest->y, chest->z);
 			data = chest->getData();
 		}
 
 		chest->checkNeighbors();
 	}
-	if (chest->n.lock() != NULL || chest->w.lock() != NULL) return;
+	if (chest->n.lock() != nullptr || chest->w.lock() != nullptr) return;
 
 
 	ChestModel *model;
-	if (chest->e.lock() != NULL || chest->s.lock() != NULL)
+	if (chest->e.lock() != nullptr || chest->s.lock() != nullptr)
 	{
 		model = largeChestModel;
 
@@ -102,7 +102,7 @@ void ChestRenderer::render(shared_ptr<TileEntity>  _chest, double x, double y, d
 	glEnable(GL_RESCALE_NORMAL);
 	//if( setColor ) glColor4f(1, 1, 1, 1);
 	if( setColor ) glColor4f(1, 1, 1, alpha);
-	glTranslatef((float) x, (float) y + 1, (float) z + 1);
+	glTranslatef(static_cast<float>(x), static_cast<float>(y) + 1, static_cast<float>(z) + 1);
 	glScalef(1, -1, -1);
 
 	glTranslatef(0.5f, 0.5f, 0.5f);
@@ -112,11 +112,11 @@ void ChestRenderer::render(shared_ptr<TileEntity>  _chest, double x, double y, d
 	if (data == 4) rot = 90;
 	if (data == 5) rot = -90;
 
-	if (data == 2 && chest->e.lock() != NULL)
+	if (data == 2 && chest->e.lock() != nullptr)
 	{
 		glTranslatef(1, 0, 0);
 	}
-	if (data == 5 && chest->s.lock() != NULL)
+	if (data == 5 && chest->s.lock() != nullptr)
 	{
 		glTranslatef(0, 0, -1);
 	}
@@ -124,12 +124,12 @@ void ChestRenderer::render(shared_ptr<TileEntity>  _chest, double x, double y, d
 	glTranslatef(-0.5f, -0.5f, -0.5f);
 
 	float open = chest->oOpenness + (chest->openness - chest->oOpenness) * a;
-	if (chest->n.lock() != NULL)
+	if (chest->n.lock() != nullptr)
 	{
 		float open2 = chest->n.lock()->oOpenness + (chest->n.lock()->openness - chest->n.lock()->oOpenness) * a;
 		if (open2 > open) open = open2;
 	}
-	if (chest->w.lock() != NULL)
+	if (chest->w.lock() != nullptr)
 	{
 		float open2 = chest->w.lock()->oOpenness + (chest->w.lock()->openness - chest->w.lock()->oOpenness) * a;
 		if (open2 > open) open = open2;