8026049: (bf) Intrinsify ByteBuffer.put{Int, Double, Float, ...} methods

Vladimir Kozlov vladimir.kozlov at oracle.com
Sat Mar 21 16:25:25 UTC 2015


If you say so, John, I am okay with BE.

Vladimir

On 3/20/15 10:21 PM, John Rose wrote:
> I'm glad to see this coming along…  The code, javadoc, and spec all look good.
>
> On Mar 20, 2015, at 3:00 PM, Vladimir Kozlov <vladimir.kozlov at oracle.com <mailto:vladimir.kozlov at oracle.com>> wrote:
>>
>>> http://cr.openjdk.java.net/~aph/unaligned.jdk.7/
>>
>> My only comment for jdk change is to use bigEndian name instead of BE in Unsafe.java. Someone from core libs should
>> look on this.
>
> Here's my take on that bit (subject of course to correction from a core-libs eng.).
>
> BE (though short), as an upper-case acronym, is more correct than mixed-case bigEndian, according to the conventions
> suggested by JLS 6.1:
>
>> Constant Names
>>
>> The names of constants in interface types should be, and final variables of class types may conventionally be, a
>> sequence of one or more words, acronyms, or abbreviations, all uppercase, with components separated by underscore "_"
>> characters. Constant names should be descriptive and not unnecessarily abbreviated. Conventionally they may be
>> any appropriate part of speech.
>>
>> Examples of names for constants include MIN_VALUE, MAX_VALUE, MIN_RADIX, and MAX_RADIX of the class Character.
>
> So I think it's OK as-is, especially since it is a private name.
>
> — John



More information about the core-libs-dev mailing list