[klippy]

I can't just use get_ent_data_entity() like that, because I want to support both 1.8.3 and previous versions.

Anyway, I updated the code and tested it on Windows listen server with AMXX 1.8.3 and Linux dedicated server with AMXX 1.8.2 (both CS 1.6). Worked flawlessly.

1.8.2 version is a bit more hardcoded though, but I made sure to support multiple games. It contains such data (copied from AMXX 1.8.3 gamedata files):

PHP Code:

new const g_SupportedMods[][modinfo_t] = {
    {"cstrike", 1408},
    {"czero", 1408},
    {"dod", 1440},
    {"valve", 1148},
    {"tfc", 2196},
    {"gearbox", 1200}
}; 


so adding more games is easy.
Should that data be read from a text file or is this good enough? Some thoughts on that please.

[ujjl]

Originally Posted by KliPPy I can't just use get_ent_data_entity() like that, because I want to support both 1.8.3 and previous versions.

The logic is the same: skip this branch
https://github.com/ValveSoftware/hal...ayer.cpp#L1504
by setting tank to NULL

PHP Code:

public Player_PostThink_Post(this) {
    if(!g_hasExecuted[this]) {
        static oldTank; oldTank = get_pdata_ent(this, m_pTank);
        set_pdata_ent(this, m_pTank, 0); //This will skip the tank re-use when controlling tank
        ExecuteHamB(Ham_Player_ImpulseCommands, this);
        set_pdata_ent(this, m_pTank, oldTank); //Everything back to normal
    }
} 


Originally Posted by KliPPy Should that data be read from a text file or is this good enough? Some thoughts on that please.

I think who needs to add new offset there, will understand where to modify the script. Who do not need new offset will just be confused with extra text file.

[klippy]

My intention was to make ImpulseCommands() called every frame, even if the game decides it shouldn't be. I made what I wanted, and I am happy with it. In my opinion, the first version was all good anyway, but I made it take tanks into account.