Should these be an enum?