There's duplicated code in MappedLoaderOopIterator vs MappedWriterOopIterator, as well as in StreamedLoaderOopIterator vs StreamedWriterOopIterator:
E.g.
https://github.com/openjdk/jdk/blob/e51ccef9cb415ed31db70971bb439ca3d96c5bce/src/hotspot/share/cds/aotMappedHeapWriter.cpp#L912-L1002
vs
https://github.com/openjdk/jdk/blob/e51ccef9cb415ed31db70971bb439ca3d96c5bce/src/hotspot/share/cds/aotMappedHeapLoader.cpp#L736-L826
The duplicated code should be moved to a super class.
E.g.
https://github.com/openjdk/jdk/blob/e51ccef9cb415ed31db70971bb439ca3d96c5bce/src/hotspot/share/cds/aotMappedHeapWriter.cpp#L912-L1002
vs
https://github.com/openjdk/jdk/blob/e51ccef9cb415ed31db70971bb439ca3d96c5bce/src/hotspot/share/cds/aotMappedHeapLoader.cpp#L736-L826
The duplicated code should be moved to a super class.
- links to
-
Review(master)
openjdk/jdk/29549