From 28614b922fb77149a54da1a87bebfbc98736f296 Mon Sep 17 00:00:00 2001
From: ModMaker101 <119018978+ModMaker101@users.noreply.github.com>
Date: Sun, 8 Mar 2026 19:08:36 -0400
Subject: 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
---
 Minecraft.Client/ConnectScreen.cpp | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

(limited to 'Minecraft.Client/ConnectScreen.cpp')

diff --git a/Minecraft.Client/ConnectScreen.cpp b/Minecraft.Client/ConnectScreen.cpp
index 2cf005b6..18d50a52 100644
--- a/Minecraft.Client/ConnectScreen.cpp
+++ b/Minecraft.Client/ConnectScreen.cpp
@@ -12,12 +12,12 @@ ConnectScreen::ConnectScreen(Minecraft *minecraft, const wstring& ip, int port)
 {
 	aborted = false;
 //    System.out.println("Connecting to " + ip + ", " + port);
-    minecraft->setLevel(NULL);
+    minecraft->setLevel(nullptr);
 #if 1
 	// 4J - removed from separate thread, but need to investigate what we actually need here
     connection = new ClientConnection(minecraft, ip, port);
     if (aborted) return;
-    connection->send( shared_ptr<PreLoginPacket>( new PreLoginPacket(minecraft->user->name) ) );
+    connection->send(std::make_shared<PreLoginPacket>(minecraft->user->name));
 #else
 
     new Thread() {
@@ -45,7 +45,7 @@ ConnectScreen::ConnectScreen(Minecraft *minecraft, const wstring& ip, int port)
 
 void ConnectScreen::tick()
 {
-    if (connection != NULL)
+    if (connection != nullptr)
 	{
         connection->tick();
     }
@@ -69,7 +69,7 @@ void ConnectScreen::buttonClicked(Button *button)
     if (button->id == 0)
 	{
         aborted = true;
-        if (connection != NULL) connection->close();
+        if (connection != nullptr) connection->close();
         minecraft->setScreen(new TitleScreen());
     }
 }
@@ -80,7 +80,7 @@ void ConnectScreen::render(int xm, int ym, float a)
 
     Language *language = Language::getInstance();
 
-    if (connection == NULL)
+    if (connection == nullptr)
 	{
         drawCenteredString(font, language->getElement(L"connect.connecting"), width / 2, height / 2 - 50, 0xffffff);
         drawCenteredString(font, L"", width / 2, height / 2 - 10, 0xffffff);
-- 
cgit v1.2.3