Hi, Thanks for pushing changes!
regards, Aya Ebata 2020年2月17日(月) 11:37 Yasumasa Suenaga <suen...@oss.nttdata.com>: > On 2020/02/17 11:21, David Holmes wrote: > > Hi Yasumasa, > > > > On 17/02/2020 11:38 am, Yasumasa Suenaga wrote: > >> Hi David, > >> > >> On 2020/02/17 8:10, David Holmes wrote: > >>> On 17/02/2020 12:40 am, Aya Ebata wrote: > >>>> Hi, > >>>> > >>>> I sent OCA today. So it hasn't been approved yet. I'm waiting for > approval. > >>> > >>> For this trivial contribution an OCA is a not required, so this can be > sponsored before the OCA goes through processing. > >> > >> I didn't know that! > >> I hope it is described in "How to contribute" page [1]. > > > > Unfortunately no, it is covered in the by-laws [1], when distinguishing > a Participant from a Contributor: > > > > "A Participant may post messages to a list, submit simple patches, and > make other kinds of small contributions." > > > > "A Contributor is a Participant who has signed the Oracle Contributor > Agreement (OCA), ..." > > Thanks for your information! > > > Cheers, > > Yasumasa > > > > Cheers, > > David > > > > [1] https://openjdk.java.net/bylaws > > > > > > > >> Anyway, I will push Aya's change to jdk/jdk. > >> > >> > >> Thanks, > >> > >> Yasumasa > >> > >> > >> [1] https://openjdk.java.net/contribute/ > >> > >> > >>> Cheers, > >>> David > >>> > >>>> regards, > >>>> Aya Ebata > >>>> > >>>> > >>>> 2020年2月16日(日) 16:49 Yasumasa Suenaga <suen...@oss.nttdata.com>: > >>>> > >>>>> Hi Aya, > >>>>> > >>>>> It looks good to me. > >>>>> BTW, have you signed to OCA? If so, I will sponsor you. > >>>>> > >>>>> > >>>>> Thanks, > >>>>> > >>>>> Yasumasa > >>>>> > >>>>> > >>>>> On 2020/02/16 16:42, 江畑 彩 wrote: > >>>>>> Hi, > >>>>>> > >>>>>> I fixed typo in s.m.Unsafe and j.i.Unsafe. The changes are below. > >>>>>> Could you sponsor this, please. > >>>>>> https://bugs.openjdk.java.net/browse/JDK-8237818 > >>>>>> > >>>>>> regards, > >>>>>> Aya Ebata > >>>>>> > >>>>>> > >>>>>> > >>>>>> diff --git > a/src/java.base/share/classes/jdk/internal/misc/Unsafe.java > >>>>>> b/src/java.base/share/classes/jdk/internal/misc/Unsafe.java > >>>>>> --- a/src/java.base/share/classes/jdk/internal/misc/Unsafe.java > >>>>>> +++ b/src/java.base/share/classes/jdk/internal/misc/Unsafe.java > >>>>>> @@ -40,7 +40,7 @@ > >>>>>> * Although the class and all methods are public, use of this > class is > >>>>>> * limited because only trusted code can obtain instances of it. > >>>>>> * > >>>>>> - * <em>Note:</em> It is the resposibility of the caller to make > sure > >>>>>> + * <em>Note:</em> It is the responsibility of the caller to make > sure > >>>>>> * arguments are checked before methods of this class are > >>>>>> * called. While some rudimentary checks are performed on the > input, > >>>>>> * the checks are best effort and when performance is an > overriding > >>>>>> @@ -425,7 +425,7 @@ > >>>>>> /** > >>>>>> * Create an exception reflecting that some of the input was > >>>>> invalid > >>>>>> * > >>>>>> - * <em>Note:</em> It is the resposibility of the caller to make > >>>>>> + * <em>Note:</em> It is the responsibility of the caller to > make > >>>>>> * sure arguments are checked before the methods are called. > While > >>>>>> * some rudimentary checks are performed on the input, the > checks > >>>>>> * are best effort and when performance is an overriding > priority, > >>>>>> @@ -601,7 +601,7 @@ > >>>>>> * aligned for all value types. Dispose of this memory by > calling > >>>>>> {@link > >>>>>> * #freeMemory}, or resize it with {@link #reallocateMemory}. > >>>>>> * > >>>>>> - * <em>Note:</em> It is the resposibility of the caller to make > >>>>>> + * <em>Note:</em> It is the responsibility of the caller to > make > >>>>>> * sure arguments are checked before the methods are called. > While > >>>>>> * some rudimentary checks are performed on the input, the > checks > >>>>>> * are best effort and when performance is an overriding > priority, > >>>>>> @@ -657,7 +657,7 @@ > >>>>>> * #reallocateMemory}. The address passed to this method > may be > >>>>> null, > >>>>>> in > >>>>>> * which case an allocation will be performed. > >>>>>> * > >>>>>> - * <em>Note:</em> It is the resposibility of the caller to make > >>>>>> + * <em>Note:</em> It is the responsibility of the caller to > make > >>>>>> * sure arguments are checked before the methods are called. > While > >>>>>> * some rudimentary checks are performed on the input, the > checks > >>>>>> * are best effort and when performance is an overriding > priority, > >>>>>> @@ -719,7 +719,7 @@ > >>>>>> * If the effective address and length are (resp.) even > modulo 4 > >>>>> or 2, > >>>>>> * the stores take place in units of 'int' or 'short'. > >>>>>> * > >>>>>> - * <em>Note:</em> It is the resposibility of the caller to make > >>>>>> + * <em>Note:</em> It is the responsibility of the caller to > make > >>>>>> * sure arguments are checked before the methods are called. > While > >>>>>> * some rudimentary checks are performed on the input, the > checks > >>>>>> * are best effort and when performance is an overriding > priority, > >>>>>> @@ -781,7 +781,7 @@ > >>>>>> * If the effective addresses and length are (resp.) even > modulo 4 > >>>>> or > >>>>>> 2, > >>>>>> * the transfer takes place in units of 'int' or 'short'. > >>>>>> * > >>>>>> - * <em>Note:</em> It is the resposibility of the caller to make > >>>>>> + * <em>Note:</em> It is the responsibility of the caller to > make > >>>>>> * sure arguments are checked before the methods are called. > While > >>>>>> * some rudimentary checks are performed on the input, the > checks > >>>>>> * are best effort and when performance is an overriding > priority, > >>>>>> @@ -842,7 +842,7 @@ > >>>>>> * as discussed in {@link #getInt(Object,long)}. When the > object > >>>>>> reference is null, > >>>>>> * the offset supplies an absolute base address. > >>>>>> * > >>>>>> - * <em>Note:</em> It is the resposibility of the caller to make > >>>>>> + * <em>Note:</em> It is the responsibility of the caller to > make > >>>>>> * sure arguments are checked before the methods are called. > While > >>>>>> * some rudimentary checks are performed on the input, the > checks > >>>>>> * are best effort and when performance is an overriding > priority, > >>>>>> @@ -901,7 +901,7 @@ > >>>>>> * #allocateMemory} or {@link #reallocateMemory}. The > address > >>>>> passed > >>>>>> to > >>>>>> * this method may be null, in which case no action is taken. > >>>>>> * > >>>>>> - * <em>Note:</em> It is the resposibility of the caller to make > >>>>>> + * <em>Note:</em> It is the responsibility of the caller to > make > >>>>>> * sure arguments are checked before the methods are called. > While > >>>>>> * some rudimentary checks are performed on the input, the > checks > >>>>>> * are best effort and when performance is an overriding > priority, > >>>>>> diff --git a/src/jdk.unsupported/share/classes/sun/misc/Unsafe.java > >>>>>> b/src/jdk.unsupported/share/classes/sun/misc/Unsafe.java > >>>>>> --- a/src/jdk.unsupported/share/classes/sun/misc/Unsafe.java > >>>>>> +++ b/src/jdk.unsupported/share/classes/sun/misc/Unsafe.java > >>>>>> @@ -39,7 +39,7 @@ > >>>>>> * Although the class and all methods are public, use of this > class is > >>>>>> * limited because only trusted code can obtain instances of it. > >>>>>> * > >>>>>> - * <em>Note:</em> It is the resposibility of the caller to make > sure > >>>>>> + * <em>Note:</em> It is the responsibility of the caller to make > sure > >>>>>> * arguments are checked before methods of this class are > >>>>>> * called. While some rudimentary checks are performed on the > input, > >>>>>> * the checks are best effort and when performance is an > overriding > >>>>>> @@ -439,7 +439,7 @@ > >>>>>> * aligned for all value types. Dispose of this memory by > calling > >>>>>> {@link > >>>>>> * #freeMemory}, or resize it with {@link #reallocateMemory}. > >>>>>> * > >>>>>> - * <em>Note:</em> It is the resposibility of the caller to make > >>>>>> + * <em>Note:</em> It is the responsibility of the caller to > make > >>>>>> * sure arguments are checked before the methods are called. > While > >>>>>> * some rudimentary checks are performed on the input, the > checks > >>>>>> * are best effort and when performance is an overriding > priority, > >>>>>> @@ -471,7 +471,7 @@ > >>>>>> * #reallocateMemory}. The address passed to this method > may be > >>>>> null, > >>>>>> in > >>>>>> * which case an allocation will be performed. > >>>>>> * > >>>>>> - * <em>Note:</em> It is the resposibility of the caller to make > >>>>>> + * <em>Note:</em> It is the responsibility of the caller to > make > >>>>>> * sure arguments are checked before the methods are called. > While > >>>>>> * some rudimentary checks are performed on the input, the > checks > >>>>>> * are best effort and when performance is an overriding > priority, > >>>>>> @@ -507,7 +507,7 @@ > >>>>>> * If the effective address and length are (resp.) even > modulo 4 > >>>>> or 2, > >>>>>> * the stores take place in units of 'int' or 'short'. > >>>>>> * > >>>>>> - * <em>Note:</em> It is the resposibility of the caller to make > >>>>>> + * <em>Note:</em> It is the responsibility of the caller to > make > >>>>>> * sure arguments are checked before the methods are called. > While > >>>>>> * some rudimentary checks are performed on the input, the > checks > >>>>>> * are best effort and when performance is an overriding > priority, > >>>>>> @@ -552,7 +552,7 @@ > >>>>>> * If the effective addresses and length are (resp.) even > modulo 4 > >>>>> or > >>>>>> 2, > >>>>>> * the transfer takes place in units of 'int' or 'short'. > >>>>>> * > >>>>>> - * <em>Note:</em> It is the resposibility of the caller to make > >>>>>> + * <em>Note:</em> It is the responsibility of the caller to > make > >>>>>> * sure arguments are checked before the methods are called. > While > >>>>>> * some rudimentary checks are performed on the input, the > checks > >>>>>> * are best effort and when performance is an overriding > priority, > >>>>>> @@ -589,7 +589,7 @@ > >>>>>> * #allocateMemory} or {@link #reallocateMemory}. The > address > >>>>> passed > >>>>>> to > >>>>>> * this method may be null, in which case no action is taken. > >>>>>> * > >>>>>> - * <em>Note:</em> It is the resposibility of the caller to make > >>>>>> + * <em>Note:</em> It is the responsibility of the caller to > make > >>>>>> * sure arguments are checked before the methods are called. > While > >>>>>> * some rudimentary checks are performed on the input, the > checks > >>>>>> * are best effort and when performance is an overriding > priority, > >>>>>> > >>>>> >