In server/time.c:process_object(), the final switch statement has:
    case DETECTOR:
        move_detector(op);

    case DIRECTOR:
        if (op->stats.maxsp)
            animate_turning(op);
        return 0;
(in both 1.8.0 release and current cvs).

Is this intended to fall through to the DIRECTOR case? If so I'd
recommend a /* fall through */ comment to clarify that.

No other cases within the switch fall through, and I couldn't find
an example in the maps of a detector with maxsp, so I suspect it is
unintentional.

Hugo

_______________________________________________
crossfire mailing list
crossfire@metalforge.org
http://mailman.metalforge.org/mailman/listinfo/crossfire

Reply via email to