RFR (S): 8214850: Rename vm_operations* to vmOperations*
coleen.phillimore at oracle.com
coleen.phillimore at oracle.com
Wed Dec 5 14:27:22 UTC 2018
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.
>
> But if we're going to impose a particular naming scheme perhaps we
> should bite the bullet and bring everything into line? arguments_ext,
> os_ext, os_perf, vm_version ...
>
Yes, eventually, depending on distracting the misnaming is, and how
difficult to find the file. vm_version seems like a candidate.
thanks,
Coleen
> Cheers,
> David
>
>> Thanks,
>> Thomas
>>
>>
More information about the hotspot-dev
mailing list