> gettext("%s team already has infinite ammo for the %s!")
> As for this bit, it is / was there for debugging, but now this is issue
is
> fixed. If people wish for it to remain that might be good, since there
is
> checking code for an infinite ammo weapon, however, nobody should ever
get
> that case unless the code is changed. Again, this is my fault :-(

I see. I saw the comment above, but wouldn't an assertion be more
appropriate here? Also a message that could be something like:

"Unexpected infinite ammo for weapon %s. Please contact
wormux-dev@gna.org about this issue."


An assertion was one idea, but that would cause the game to quit, while
getting such an error is not  really a important problem  due to the error
checking. The game shouldn't be affected by such a box appearing, though if
one does I would like to know about it. I believe I will convert it to ask
for a bug report, but unless anyone else thinks otherwise, I would rather
not have the game assert and quit due to that reason alone.

robo210
_______________________________________________
Wormux-dev mailing list
Wormux-dev@gna.org
https://mail.gna.org/listinfo/wormux-dev

Répondre à