This is fine for now, ideally we should also handle the empty case