This looks really strange, and caught my eye. It's syntactically identical to `run_mode == 2 && (safemodeveh || !u.controlling_vehicle)`, and I'm not sure if that's the actual intent of the test.