shouldn't it be an assert?