From 3acc39095ea668dc4fe2b2f907a8cdec4be3a6f9 Mon Sep 17 00:00:00 2001
From: Fabio Alessandrelli <fabio.alessandrelli@gmail.com>
Date: Fri, 30 Apr 2021 15:52:29 +0200
Subject: [PATCH] [Net] Fix bogus FileAccessNetwork deconstructor.

Now correctly erases old instances.
The code will likely need overhaul anyway to be usable.
It doesn't apply to editor runs, there's a bunch of inconsistencies on
how to clients are handled, and I don't really understand why multiple
instances are created for a single client/server.
---
 core/io/file_access_network.cpp | 1 -
 1 file changed, 1 deletion(-)

diff --git a/core/io/file_access_network.cpp b/core/io/file_access_network.cpp
index 307004b1c27..cb38ac09284 100644
--- a/core/io/file_access_network.cpp
+++ b/core/io/file_access_network.cpp
@@ -487,7 +487,6 @@ FileAccessNetwork::~FileAccessNetwork() {
 
 	FileAccessNetworkClient *nc = FileAccessNetworkClient::singleton;
 	nc->lock_mutex();
-	id = nc->last_id++;
 	nc->accesses.erase(id);
 	nc->unlock_mutex();
 }