You should make the nullret stuff static, than it works from const.

And why has that not been static since its introduction, there is really no point in having each overmap there own nullret object. Alternatively you can make it mutable.

A `const oter_id &overmap::ter(...) const` will only allow getting and not allow setting, too.