Update func_vehicle for multiplayer

[UnrealKaraulov]

• new CVar: mp_legacy_vehicle_block.

Video desc:

• https://www.youtube.com/watch?v=2UbD5JGc7xs
  

• https://www.youtube.com/watch?v=yQ1E9JvosDA (Save m_pDriver)
  

[UnrealKaraulov]

@s1lentq Хотим на сервер карт с машинами залить типа big_city2 всем нравятся карты, даже пусть физика у машин не самая лучшая)))

но вот некоторые проблемы сильно мешают использовать такие карты на сервере!

В общем на картах с машинами невозможно играть по двум причинам...

1 - это убийство своих.
2 - убийство врагов без отображения кила.

Тут все решается.
Записал два видео.

[UnrealKaraulov]

Так же если заметили отключил m_pDriver = nullptr когда игрок отключает управление машиной, что это дает, видно на последнем видео.

[SergeyShorokhov]

Так же если заметили отключил m_pDriver = nullptr когда игрок отключает управление машиной, что это дает, видно на последнем видео.

I am not sure that these changes will be the correct behavior for the entity. Because, in actuality, the entity ALWAYS does not have a driver after the player has left the vehicle. And this is the correct behavior for GameDLL.

If you want the gameplay to consider a kill from the vehicle after the player has left it, you need to handle this as a specific case.

[UnrealKaraulov]

Так же если заметили отключил m_pDriver = nullptr когда игрок отключает управление машиной, что это дает, видно на последнем видео.

I am not sure that these changes will be the correct behavior for the entity. Because, in actuality, the entity ALWAYS does not have a driver after the player has left the vehicle. And this is the correct behavior for GameDLL.

If you want the gameplay to consider a kill from the vehicle after the player has left it, you need to handle this as a specific case.

In multiplayer need known who is latest driver for give frag him.
Now fixed default behavior, added check for behavior cvar.

Ну смысл данного квара что бы сделать все иначе чем в оригинале, в общем если я правильно понял включил проверку на mp_legacy_vehicle_block в данные изменения, теперь все устраивает ? :(

Так же заметил что при выходе игрока m_pDriver не устанавливался в NULLPTR и добавил это в ClientDisconnected. Правильно ли я сделал? Или это избыточная проверка и ничего не случилось бы?

(Допустим игрок пишет disconnect во время разгона, и автомобиль убивает что-нибудь, m_pDriver все еще будет указывать на этого игрока. И не знаю к чему это приведет, по этому на всякий случай добавил очистку в ClientDisconnected, хотя ее изначально нет в GameDLL)

[hajimura]

Допустим игрок пишет disconnect во время разгона, и автомобиль убивает что-нибудь, m_pDriver все еще будет указывать на этого игрока. И не знаю к чему это приведет, по этому на всякий случай добавил очистку в ClientDisconnected, хотя ее изначально нет в GameDLL

I think, there shouldn't be a problem with it. Just in case we could add has_disconnected check

[UnrealKaraulov]

@hajimura but m_pDriver is not nullptr! This check not works in this case, because m_pDriver not nullptr if player disconnected.

@wopox1337 Now all okay or need some more changes ?

[hajimura]

but m_pDriver is not nullptr!

we could add has_disconnected check

[UnrealKaraulov]

we could add has_disconnected check

has_disconnected not require valid CBasePlayer struct ? or this class not deleted after player disconnect?

[SergeyShorokhov]

Waiting for @s1lentq review

[hajimura]

this class not deleted after player disconnect?

Yep. Slot that was released after player disconnect still has CBasePlayer class, just sets FL_DORMANT flag (https://github.com/s1lentq/ReGameDLL_CS/blob/master/regamedll/dlls/cbase.cpp#L844-L854) and he updates after player connect

[UnrealKaraulov]

@hajimura if player disconnected, but in this time, connected another player ?

yes, this is very rary case, i think your variant is better

[hajimura]

🚢

[UnrealKaraulov]

I think all okay

[SergeyShorokhov]

The new functionality needs to be introduced in a different way. At least this way:

  CFuncVehicle::m_hLastDriver;
  EntityHandle<CBaseEntity> m_hLastDriver;

and need to think better about the CVar name