nit: should do nothing when...