shouldn't it be const?