found a bug recently introduced, doing a comparison instead that an assignment