Assertion fails when botdontshoot is enabled

[josemam]

As soon as the first bot joins the game if botdontshoot is enabled in the config file, or as soon as it is turned on when there are bots in the game, this message is shown and hlds crashes:

[jk_botti] [ASSERT] 'pBot.pBotEnemy == NULL' : 'bot_combat.cpp' : 'line 756'

I tried commenting out the line (had to, in order to test for #3) and the server seemed to run fine, or at least nothing weird was observed in the server log.

[djearthquake]

Thanks for acknowledging. How easy it is to forget and not notice such details. I needed to make the bots ignore humans to do some testing where X number of players need to be present. For the last several years I have had a scrolling message vote_ignore to make bots ignore humans. Many crashes recently using this newer code. Are there any plans on changing the source or just keeping this indexed here?

server console (cmdline "jk_botti botdontshoot 1")
[jk_botti] botdontshoot ENABLED
[jk_botti] [ASSERT] 'pBot.pBotEnemy == NULL' : 'bot_combat.cpp' : 'line 756'
Trace/breakpoint trap

[jkivilin]

Well.. to be honest, I have not played HL for years and jk_botti is not high in my priority list (of hobby programming). I made commit to fix this issue through github editing interface, completely untested. I hope it works.

[djearthquake]

Observer works fine so that will due. I'll focus on this over the Winter or when I get free time.