shouldn't we default to None?