This reverts commit 15450a6.

This reverts commit e5ddbd9.

This change ensures that the iterator values used by `std::distance`
is correct.  Having the emplace within leads to the possibility of
`m_Entities.begin()` being invalidated due to reallocations.

(cherry picked from commit 89bd4d7)

…dders#1859)

The extended gameconfig format reads the master gameconf file twice;
once each for the base engine and actual engine.  The file list
isn't checked for duplicates, so 'common.games.txt' is loaded in
twice, resulting in any 'common' file values potentially overriding
values listed under '#default' in other files due to
bShouldBeReadingDefault.  This happens in the case when matching
game versions by CRC (such as public game branches).

Required for alliedmodders#1857.

(cherry picked from commit 34c8220)

On NMRiH and possibly other games that use the Maphacks system,
entries that are modified using that system are rendered with tab
characters stripped out - see CNMRiHMapHackManager::GetEntDataString.

That results in there being no separators at all between keys and
values, as Maphacks receives the serialized string from Entity Lump
Manager.

This commit changes the key / value separator character to use
spaces instead.

This discovery upsets me greatly.

Fixes alliedmodders#1833.

(cherry picked from commit ef7d3ab)

(cherry picked from commit 0d61792)

)

(cherry picked from commit 27b1817)

Addie said:
test the signatures:
\x55\x89\xE5\x57\x56\x53\x81\xEC\x8C\x01\x00\x00\x8B\x55\x08
or
digby's \x55\x89\xE5\x57\x56\x53\x81\xEC\x8C\x01\x00\x00\x8B\x55\x2A\x65\xA1

\x55\x89\xE5\x57\x56\x53\x81\xEC\x8C\x01\x00\x00\x8B\x55\x08 fixed the issue for linux server.

(cherry picked from commit 2dcee81)

(cherry picked from commit 6e839a9)

(cherry picked from commit a0eb6a9)

Note: Currently only works on beta version of game
(cherry picked from commit 397b70a)

(cherry picked from commit 693e584)

Adds the LookupAttachment signature for Nuclear Dawn

(cherry picked from commit a9a1939)

…eplyToTargetError COMMAND_TARGET_AMBIGUOUS.

Explanation:
There are two clients in the server, one named gene, the other one "Ene ~special characters~".
An admin issues "sm_slay Ene" and gets following error message: More than one client matched the given pattern.
What this hack will do is: Use GetCmdArg(0, ...); to get the command name "sm_slay".
Use GetCmdArgString(...); to get the arguments supplied to the command.
Use GetLastProcessTargetString(...); (which was implemented in this commit) to retrieve the arguments that were passed to the last ProcessTargetString call.
It will then pass this data to the DynamicTargeting plugin through its AmbiguousMenu native.
The plugin will open up a menu on the client and list all targets which match the pattern that was supplied to ProcessTargetString.
If the client selects a menu entry, FakeClientCommand will be used to re-execute the command with the correct target.

# Conflicts:
#	.github/workflows/ci.yml
#	AMBuildScript
#	core/AMBuilder
#	core/UserMessages.cpp
#	core/UserMessages.h
#	extensions/dhooks/DynamicHooks/hook.cpp
#	gamedata/core.games/common.games.txt
#	gamedata/sdkhooks.games/engine.ep2v.txt
#	gamedata/sdktools.games/game.nucleardawn.txt
#	gamedata/sdktools.games/game.tf.txt
#	gamedata/sm-tf2.games.txt
#	plugins/AMBuilder
#	sourcepawn