Can we just implement this as defined in the spec?

inEffect = activeTime != null;

We want to keep players in this list for anything which is current, or in effect.