[Pierre-Marie Baty]

In the barebones PB2.6 source code, the bot names array was a dynamically allocated linked list, that could in theory grow as far as there were names to add to it. My first go at fixing the memory leaks and the slowmo issue was to get rid of as much dynamic memory as possible and turn all these linked lists into static arrays. I never thought one could need more than 100 botnames (since 32 players on a server is the maximum you can go anyway).

If you want to make it dynamic, I'm against linked lists. Better count the number of lines in botnames.txt and allocate once and for all enough memory for the bot names, and then fill the array. You mustn't forget to do the same for the used bot names array. And NEITHER must you forget to free them all along with the rest of the memory the bot uses (in ServerDeactivate() if I remember well).