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

diff --git a/Minecraft.World/FileOutputStream.cpp b/Minecraft.World/FileOutputStream.cpp
index 557ba812..58a0ecd9 100644
--- a/Minecraft.World/FileOutputStream.cpp
+++ b/Minecraft.World/FileOutputStream.cpp
@@ -24,20 +24,20 @@ FileOutputStream::FileOutputStream(const File &file) : m_fileHandle( INVALID_HAN
 		file.getPath().c_str() , // file name
 		GENERIC_WRITE, // access mode
 		0, // share mode // TODO 4J Stu - Will we need to share file? Probably not but...
-		NULL, // Unused
+		nullptr, // Unused
 		OPEN_ALWAYS , // how to create
 		FILE_ATTRIBUTE_NORMAL , // file attributes
-		NULL // Unsupported
+		nullptr // Unsupported
 		);
 #else
 	m_fileHandle = CreateFile(
 		wstringtofilename(file.getPath()) , // file name
 		GENERIC_WRITE, // access mode
 		0, // share mode // TODO 4J Stu - Will we need to share file? Probably not but...
-		NULL, // Unused
+		nullptr, // Unused
 		OPEN_ALWAYS , // how to create
 		FILE_ATTRIBUTE_NORMAL , // file attributes
-		NULL // Unsupported
+		nullptr // Unsupported
 		);
 #endif
 
@@ -61,14 +61,14 @@ void FileOutputStream::write(unsigned int b)
 {	
 	DWORD numberOfBytesWritten;
 
-	byte value = (byte) b;
+	byte value = static_cast<byte>(b);
 
 	BOOL result = WriteFile(
 		m_fileHandle, // handle to file
 		&value, // data buffer
 		1, // number of bytes to write
 		&numberOfBytesWritten, // number of bytes written
-		NULL // overlapped buffer
+		nullptr // overlapped buffer
 		);
 
 	if( result == 0 )
@@ -93,7 +93,7 @@ void FileOutputStream::write(byteArray b)
 		&b.data, // data buffer
 		b.length, // number of bytes to write
 		&numberOfBytesWritten, // number of bytes written
-		NULL // overlapped buffer
+		nullptr // overlapped buffer
 		);
 
 	if( result == 0 )
@@ -123,7 +123,7 @@ void FileOutputStream::write(byteArray b, unsigned int offset, unsigned int leng
 		&b[offset], // data buffer
 		length, // number of bytes to write
 		&numberOfBytesWritten, // number of bytes written
-		NULL // overlapped buffer
+		nullptr // overlapped buffer
 		);
 
 	if( result == 0 )