RFR - 8132734: java.util.jar.* changes to support multi-release jar files
Paul Sandoz
paul.sandoz at oracle.com
Fri Jan 22 09:26:44 UTC 2016
> On 22 Jan 2016, at 01:39, Mandy Chung <mandy.chung at oracle.com> wrote:
>
>
>> On Jan 21, 2016, at 3:49 PM, Steve Drach <steve.drach at oracle.com> wrote:
>>
>>>> I suspected this is a bike shed candidate. I think Release._9 is nicer and it conveys the same information in a less cluttered way than Release.RELEASE_9.
>>> Yes a bike shed, I'm just saying that Release._9 looks odd/inconsistent when we have SourceVersion.RELEASE_9 elsewhere. Maybe there has been discussion on this topic already. With a static import then RELEASE_9 isn't too bad.
>>
>> I’ll leave this as an open issue for awhile in case I get another reviewer that feels as strongly about it you do, or as I do.
>>
>
> I only started looking at some files on the webrev. Release._9 catches my attention too and it looks very odd. I think RELEASE_9 is a much better constant name than _9.
>
While there is a some naming activity over what to call such constants, i think the use a ‘_’ as the first character of a public API artefact should be strongly discouraged, such usages are more commonly associated with internal artefacts or generated code and using such a style for public artefacts sets a “bad" precedence IMO (first use in the Java APIs AFAICT). There needs to be a really strong justification for such public use and at the moment i don’t see one here.
Paul.
More information about the core-libs-dev
mailing list