Most of the CDS code refers to a memory blocks inside a CDS file as "regions", represented by the type FileMapRegion.

We have this legacy API:

  FileMapRegion* FileMapInfo::space_at(int i)

which looks awkward:

  FileMapRegion* si = space_at(MetaspaceShared::bm);
  if (si->mapped_base() != NULL) {
    return si->mapped_base();
  }

=====================

Proposal:

rename API to

FileMapRegion* FileMapInfo::region_at(int i)

Change the relevant code to

  FileMapRegion* r = space_at(MetaspaceShared::bm);
  if (r->mapped_base() != NULL) {
    return r->mapped_base();
  }