shouldn't this be const?