should be a const?