fix default player colors, we should not reference I outside the loop

[barbeque-squared]

The problematic lines were referencing I outside the loop. I assume this gets compiled differently on different platforms, because it never triggered for me (Linux). To reproduce I had to hardcode the deleted lines to read from P12.

Either way, I've made the default colors an array of numbers that just happen to be in the desired order. Downside is it's one more thing to change if we ever increase IMaxPlayerCount but that's not happening soon anyway. And at least I put the array directly below it,

I will tag some people to test this once the artifacts are built, because I cannot verify if this fixes it for them as well.

fixes #958