[crossfire] directing detectors?

hv at crypt.org hv at crypt.org
Sun Jan 15 08:38:32 CST 2006


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



More information about the crossfire mailing list