Fix Wunused-variable warnings (#713)

[AMS21]

Mostly resorted to commenting out variables in case they might still be useful to someone someway.

What confused me is the bit in IPureClient::Disconnect(). This almost looks like we're missing some cleanup here.

[Xottab-DUTY]

Uh-oh, this conflicts with #1425. @Masterkatze is working on warnings task, and a CMake refactoring.
So, I'm okay if you both will work on that, but we need to synchronize the work, so that we don't spend time on duplicating changes :)

[AMS21]

Oh very good point. Thanks for pointing that out :)

[Xottab-DUTY]

What confused me is the bit in IPureClient::Disconnect(). This almost looks like we're missing some cleanup here.

From what I heard, there's a ton of stack overflows and other bugs.
From what I know, they are actually USED to turn multiplayer servers down, exploited for cheats, etc.

(only on Windows, Linux builds don't support multiplayer at all currently)

[AMS21]

Removed all changes to files inside src/Layers as to not conflict with #1425

[AMS21]

What confused me is the bit in IPureClient::Disconnect(). This almost looks like we're missing some cleanup here.

From what I heard, there's a ton of stack overflows and other bugs. From what I know, they are actually USED to turn multiplayer servers down, exploited for cheats, etc.

(only on Windows, Linux builds don't support multiplayer at all currently)

Oh no but I don't think fixing these problems is a high priority.