RFR (S): 8214850: Rename vm_operations* to vmOperations*
Thomas Schatzl
thomas.schatzl at oracle.com
Thu Dec 6 12:56:31 UTC 2018
Hi Coleen, David,
On Wed, 2018-12-05 at 09:27 -0500, coleen.phillimore at oracle.com wrote:
> This change looks good to me. I type this name wrong every time.
>
> On 12/5/18 6:08 AM, David Holmes wrote:
> > Hi Thomas,
> >
> > On 5/12/2018 8:47 pm, Thomas Schatzl wrote:
> > > Hi all,
> > >
> > > can I have reviews for this change that renames the
> > > vm_operations files to vmOperations?
> > >
> > > This has been requested in the JDK-8214791 review but I did not
> > > want to merge these.
> > >
> > > CR:
> > > https://bugs.openjdk.java.net/browse/JDK-8214850
> > > Webrev:
> > > http://cr.openjdk.java.net/~tschatzl/8214850/webrev/
> > > Testing:
> > > build on all (Oracle) platforms
> >
> > I'm not a fan of gratuitous renames as it makes it harder to query
> > the history of a file.
> >
>
> hg mv will keep the history of the file moved.
>
>
> > But this seems to be an accurate renaming so in that sense this is
> > Reviewed.
Thanks for your reviews.
Thomas
More information about the hotspot-dev
mailing list