Yeah, but if someone wrote cEvilDoctor.ChangeRoom(..., evilDoctorLoop) and there is no evilDoctorLoop, you'd want the user to know that he made a mistake. If you don't crash on debug and just do nothing, he might not realize why his evil doctor doesn't animate properly. He'll figure out eventually that evilDoctorLoop should be 4 and not 5, but you can save him some head scratching by showing him the error.
Yeah, but that loop parameter is only for sake of direction, FaceLocation doesn't explode if you try to turn the character to a direction for which it doesn't have a loop. The point was to save one the trouble of running two commands, thus my reticence to throw errors.
I could enforce the use of an enum for directions (eDirectionUp,eDirectionLeft...), but then there could be people doing weird custom things.
Or screw them ?
I've worked with it on my last game (9 months in) and fixed all the bugs I found. Nobody reported the bug you found and I don't remember seeing it. I just re-checked and it looks fine to me. If you can get latest from svn, and check if you still have this issue so that I could fix it (send me a screenshot), I'd appreciate it.
Gotta be the theme I'm using then.
But now that I tried again I noticed another problem, scintilla seems to occasionally break (no formatting, no lines and with an asterisk even though it's not modified) when moving the pane onto different places, did that ever happen to you ?
And while we're on the subject, do you intend to check in your changes to the codebase? It would be a shame if you don't...
Do you mean the official codebase or mine own on github ? I wanted to sort out this little dilemma about changeroom before committing.