RFR: 8197852: Move G1DefaultPolicy into G1Policy (original) (raw)

Thomas Schatzl thomas.schatzl at oracle.com
Wed Mar 7 11:47:34 UTC 2018

Hi,

On Wed, 2018-03-07 at 11:39 +0100, Erik Helin wrote:

On 03/07/2018 11:04 AM, Thomas Schatzl wrote: > Hi, > > On Mon, 2018-03-05 at 16:02 +0100, Erik Helin wrote: > > Hi all, > > > > this fairly small patch re-unites G1Policy and G1DefaultPolicy, > > there > > is no need for having both classes. With this patch we only have > > G1Policy (and of course G1CollectorPolicy as the little > > CollectorPolicy shim). > > > > Issue: > > https://bugs.openjdk.java.net/browse/JDK-8197852 > > > > Patch: > > http://cr.openjdk.java.net/~ehelin/8197852/00/ > > > > Testing: > > - make run-test-tier1 on Linux, Windows, Mac, Solaris (SPARC and > > x86- > > 64) > > in g1Policy.hpp, if you update the copyright notice, please use > the correct year (2018) :)

Oops, my bad, I will fix before I push :) > Is there a way to move g1DefaultPolicy.hpp into g1Policy.hpp while > retaining history? I.e. delete the latter first, and then rename? Yep, the commit does that, it is just not reflected in the webrev. The "trick" is to hg mv g1DefaultPolicy.hpp g1Policy.hpp and then make the changes, that keeps the history.

okay, thanks for the notice!

Thomas

More information about the hotspot-gc-dev mailing list