not a big deal, but did you mean for this to be const?