don't return in a constructor