On Tue, 24 Aug 2021 17:04:01 GMT, Valerie Peng <valer...@openjdk.org> wrote:

>> Weijun Wang has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   best practice for reading DEFAULT, switch expression, etc
>
> src/java.base/share/classes/sun/security/util/DerValue.java line 319:
> 
>> 317: 
>> 318:     /**
>> 319:      * Wraps a byte array at a single DerValue.
> 
> nit: at=>as

There is already a pkg private constructor with exactly same list of arguments. 
What is the reason for adding a new public static method vs change the pkg 
private constructor to public?

-------------

PR: https://git.openjdk.java.net/jdk/pull/5221

Reply via email to