In D1064#43977, @elexis wrote:

you mean replacing if (n > 1) with if (a < n - 1) check

These 3 methods (AddUInt16, AddUInt32, GetFromBuffer) are copied from STK's BareNetworkString. I didn't copy the entire class since I tried to extract the necessary code only, hoping to replace these general methods with pyrogenesis' own or library counterparts.

Add an explicit "hostUsername" param into the lobby game record instead of parsing the first player name (see ed7ccf8)

Attempt to bind STUN-enabled client to a couple of subsequent ports if the default one is occupied (see d3eb265).

Merged @elexis' changes from https://github.com/AlexanderOlkhovskiy/0ad/pull/1

Refactoring:

• merge duplicate hole-punching-messages-sending code (1c0666d + 936d3b8)
• replace duplicate code blocks with a loop (29d5116)
• replace SDL_Delay with std::this_thread::sleep_for in StunClient.cpp, get rid of sdl dependency for the 'network' project (ee4e8b1)

and a fix:

• send hole-punching messages on client only when stun is enabled (660cba1)

Merged the latest master from the github repo (rP19507) and fixed Windows build on it (one more #undef was needed: bc492f1)

In D214#9469, @Gallaecio wrote:

<...> what types of issues it manages to find