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