shouldnt this be an assert?