should be const?