Reduce code nesting in Recv() method

[ChillerDragon]

Also replace the variable name Result with Error Since UnpackPacket() returns zero on success.

This red a bit funny:

if(!Result)
{
	// do something with the result
}

It now is:

if(Error)
	continue;

// do something with the result

[Dune-jr]

I don't know, this isn't a boolean. result being used to describe a return value that is an error when non zero is pretty common.
Whatever, that code is fine as well, but this sort of "let's move this code around in an arguably better or not fashion" causes a lot of unnecessary merge conflicts for a repository that has 610 forks.

[ChillerDragon]

I don't know, this isn't a boolean. result being used to describe a return value that is an error when non zero is pretty common.

Oh I did not realize that. I assumed that it was objectively an improvement.

Whatever, that code is fine as well, but this sort of "let's move this code around in an arguably better or not fashion" causes a lot of unnecessary merge conflicts for a repository that has 610 forks.

I agree with that mindset in general. I am also a big fan of merge conflict driven development and caring about fork experience. I just assumed nobody touched the core networking code so it shouldn't cause any git conflicts.

offtopic:
Out of curiosity do you know any fork that is merging with teeworlds? I know @fokkonaut used to do that but he stopped doing so at some point. I assume you @Dune-jr still merge the gamer client? I was browsing the list of forks and none of them looked active but maybe they have some hidden branches or private repositories or they are not github forks. I know heinrich has a few mods in the branches of his fork but as far as I know none of those were touched in the last 10 years. I could name a few active ddnet forks but except assuming gamer client is still merging I can not come up with a single teeworlds fork that is still merging. That of course is no argument and we should still avoid introducing useless code diffs.

[heinrich5991]

https://github.com/teeworlds/teeworlds/pull/3241/files?w=1 for reviewing a smaller diff.