Some regulations...

src/protocol/AcknowledgePacket.php

@@ -49,13 +49,12 @@ protected function encodePayload(PacketSerializer $out) : void{
 					if($start === $last){
 						$payload .= chr(self::RECORD_TYPE_SINGLE);
 						$payload .= Binary::writeLTriad($start);
-						$start = $last = $current;
 					}else{
 						$payload .= chr(self::RECORD_TYPE_RANGE);
 						$payload .= Binary::writeLTriad($start);
 						$payload .= Binary::writeLTriad($last);
-						$start = $last = $current;
 					}
+					$start = $last = $current;			   
 					++$records;
 				}
 			}

src/protocol/Datagram.php

@@ -50,7 +50,7 @@ protected function encodePayload(PacketSerializer $out) : void{
 	/**
 	 * @return int
 	 */
-	public function length(){
+	public function length(): int{
 		$length = self::HEADER_SIZE;
 		foreach($this->packets as $packet){
 			$length += $packet->getTotalLength();

src/protocol/OfflineMessage.php

@@ -32,14 +32,14 @@ abstract class OfflineMessage extends Packet{
 	 * @return void
 	 * @throws BinaryDataException
 	 */
-	protected function readMagic(BinaryStream $in){
+	protected function readMagic(BinaryStream $in): void{
 		$this->magic = $in->get(16);
 	}
 
 	/**
 	 * @return void
 	 */
-	protected function writeMagic(BinaryStream $out){
+	protected function writeMagic(BinaryStream $out): void{
 		$out->put($this->magic);
 	}
 

src/protocol/Packet.php

@@ -20,7 +20,7 @@
 
 abstract class Packet{
 	/** @var int */
-	public static $ID = -1;
+	public static int $ID = -1;
 
 	public function encode(PacketSerializer $out) : void{
 		$this->encodeHeader($out);

src/server/Server.php

@@ -397,7 +397,7 @@ public function openSession(ServerSession $session) : void{
 
 	private function checkSessions() : void{
 		if(count($this->sessions) > 4096){
-			foreach($this->sessions as $sessionId => $session){
+			foreach($this->sessions as $session){
 				if($session->isTemporary()){
 					$this->removeSessionInternal($session);
 					if(count($this->sessions) <= 4096){

src/server/ServerSocket.php

@@ -59,9 +59,10 @@ public function disableBroadcast() : bool{
 	}
 
 	/**
-	 * @param string $source reference parameter
-	 * @param int    $port reference parameter
+	 * @param string|null $source reference parameter
+	 * @param int|null $port reference parameter
 	 *
+	 * @return string|null
 	 * @throws SocketException
 	 */
 	public function readPacket(?string &$source, ?int &$port) : ?string{

src/utils/ExceptionTraceCleaner.php

@@ -53,16 +53,12 @@ public function getTrace(int $start = 0, ?array $trace = null) : array{
 		for($i = $start; isset($trace[$i]); ++$i, ++$j){
 			$params = "";
 			if(isset($trace[$i]["args"]) or isset($trace[$i]["params"])){
-				if(isset($trace[$i]["args"])){
-					$args = $trace[$i]["args"];
-				}else{
-					$args = $trace[$i]["params"];
-				}
-				foreach($args as $name => $value){
+				$args = $trace[$i]["args"] ?? $trace[$i]["params"];
+				foreach($args as $value){
 					$params .= (is_object($value) ? get_class($value) . " " . (method_exists($value, "__toString") ? $value->__toString() : "object") : gettype($value) . " " . @strval($value)) . ", ";
 				}
 			}
-			$messages[] = "#$j " . (isset($trace[$i]["file"]) ? $this->cleanPath($trace[$i]["file"]) : "") . "(" . (isset($trace[$i]["line"]) ? $trace[$i]["line"] : "") . "): " . (isset($trace[$i]["class"]) ? $trace[$i]["class"] . (($trace[$i]["type"] === "dynamic" or $trace[$i]["type"] === "->") ? "->" : "::") : "") . $trace[$i]["function"] . "(" . substr($params, 0, -2) . ")";
+			$messages[] = "#$j " . (isset($trace[$i]["file"]) ? $this->cleanPath($trace[$i]["file"]) : "") . "(" . ($trace[$i]["line"] ?? "") . "): " . (isset($trace[$i]["class"]) ? $trace[$i]["class"] . (($trace[$i]["type"] === "dynamic" or $trace[$i]["type"] === "->") ? "->" : "::") : "") . $trace[$i]["function"] . "(" . substr($params, 0, -2) . ")";
 		}
 
 		return $messages;