So @spastorino and @Nashenas88 have been on this awesome kick of making all "index-like things" use newtype_index!. In #45825, I made RegionVid implement Idx, but I didn't use newtype_index! because -- well, because dang it that PR is long enough. It'd be nice to do.
(I think btw we should shoot for not using public fields when we do these conversions -- I'd like to be able to transparently start using Nonzero to make Option<I>, as described in #45842.)
So @spastorino and @Nashenas88 have been on this awesome kick of making all "index-like things" use
newtype_index!. In #45825, I madeRegionVidimplementIdx, but I didn't usenewtype_index!because -- well, because dang it that PR is long enough. It'd be nice to do.(I think btw we should shoot for not using public fields when we do these conversions -- I'd like to be able to transparently start using
Nonzeroto makeOption<I>, as described in #45842.)