RFR (XXXS): 8160874: Remove typo in G1InitalMarkClosures

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

RFR (XXXS): 8160874: Remove typo in G1InitalMarkClosures

Thomas Schatzl
Hi all,

  can I have reviews for this change that fixes a typo in a class name
(G1InitalMarkClosure ->  G1InitialMarkClosure).

CR:
https://bugs.openjdk.java.net/browse/JDK-8160874
Webrev:
http://cr.openjdk.java.net/~tschatzl/8160874/webrev/
Testing:
jprt

I would like to invoke the "trivial change" clause for this to avoid
too much fuss about this.

Thanks,
  Thomas
Reply | Threaded
Open this post in threaded view
|

Re: RFR (XXXS): 8160874: Remove typo in G1InitalMarkClosures

Stefan Karlsson
Looks good.

StefanK

On 2017-02-22 11:03, Thomas Schatzl wrote:

> Hi all,
>
>   can I have reviews for this change that fixes a typo in a class name
> (G1InitalMarkClosure ->  G1InitialMarkClosure).
>
> CR:
> https://bugs.openjdk.java.net/browse/JDK-8160874
> Webrev:
> http://cr.openjdk.java.net/~tschatzl/8160874/webrev/
> Testing:
> jprt
>
> I would like to invoke the "trivial change" clause for this to avoid
> too much fuss about this.
>
> Thanks,
>   Thomas
>
Reply | Threaded
Open this post in threaded view
|

Re: RFR (XXXS): 8160874: Remove typo in G1InitalMarkClosures

Stefan Johansson


On 2017-02-22 12:10, Stefan Karlsson wrote:
> Looks good.
>
+1, a true XXXS :)
StefanJ

> StefanK
>
> On 2017-02-22 11:03, Thomas Schatzl wrote:
>> Hi all,
>>
>>   can I have reviews for this change that fixes a typo in a class name
>> (G1InitalMarkClosure ->  G1InitialMarkClosure).
>>
>> CR:
>> https://bugs.openjdk.java.net/browse/JDK-8160874
>> Webrev:
>> http://cr.openjdk.java.net/~tschatzl/8160874/webrev/
>> Testing:
>> jprt
>>
>> I would like to invoke the "trivial change" clause for this to avoid
>> too much fuss about this.
>>
>> Thanks,
>>   Thomas
>>

Reply | Threaded
Open this post in threaded view
|

Re: RFR (XXXS): 8160874: Remove typo in G1InitalMarkClosures

Thomas Schatzl
Hi Stefan and Stefan,

On Wed, 2017-02-22 at 13:54 +0100, Stefan Johansson wrote:
>
> On 2017-02-22 12:10, Stefan Karlsson wrote:
> >
> > Looks good.
> >
> +1, a true XXXS :)
> StefanJ
> >
> > StefanK

  thanks for your reviews.

Thomas