Isn't it better to just remove the ! operator here, and not swapping the returns?