On Wed, 9 Sep 2020 18:41:25 GMT, Lois Foltan <github.com+66382410+lfol...@openjdk.org> wrote:
>> Ioi Lam has updated the pull request with a new target base due to a merge >> or a rebase. The incremental webrev excludes >> the unrelated changes brought in by the merge/rebase. The pull request >> contains six additional commits since the last >> revision: >> - Removed TODO comment referring to JBS issue >> - Merge branch 'master' into 8244778-archive-full-module-graph >> - fixed trailing spaces >> - Renamed ModuleEntry::write_growable_array >> - Update to latest repo (JDK-8251557); added comments >> - 8244778: Archive full module graph in CDS > > src/hotspot/share/oops/instanceKlass.cpp line 2550: > >> 2548: // clear _nest_host to ensure re-load at runtime >> 2549: _nest_host = NULL; >> 2550: _package_entry = NULL; // TODO -- point it to the archived >> PackageEntry (JDK-8249262) > > Would you consider removing this comment? I tend not to like TODO comments > since sometimes the open enhancement > remains unaddressed. I removed the comments. ------------- PR: https://git.openjdk.java.net/jdk/pull/80