should you not throw an exception here?