RFR (XS) 8213246: Fix typo in vmTestbase failuire to failure

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

RFR (XS) 8213246: Fix typo in vmTestbase failuire to failure

JC Beyler
Hi all,

Could I get a review for a trivial typo fix:
Reply | Threaded
Open this post in threaded view
|

Re: RFR (XS) 8213246: Fix typo in vmTestbase failuire to failure

Mikael Vidstedt-3

Looks good.

Cheers,
Mikael - not a Reviewer

On Nov 2, 2018, at 4:05 PM, JC Beyler <[hidden email]> wrote:

Hi all,

Could I get a review for a trivial typo fix:

Reply | Threaded
Open this post in threaded view
|

Re: RFR (XS) 8213246: Fix typo in vmTestbase failuire to failure

Daniel D. Daugherty
In reply to this post by JC Beyler


On 11/2/18 7:05 PM, JC Beyler wrote:
Hi all,

Could I get a review for a trivial typo fix:
 
test/hotspot/jtreg/vmTestbase/nsk/share/native/nsk_tools.cpp
    No comments.

test/hotspot/jtreg/vmTestbase/nsk/stress/stack/stack001.java
    No comments.

Thumbs up!

Dan



Reply | Threaded
Open this post in threaded view
|

Re: RFR (XS) 8213246: Fix typo in vmTestbase failuire to failure

Igor Ignatyev
In reply to this post by Mikael Vidstedt-3
LGTM

-- Igor - a Reviewer

On Nov 2, 2018, at 4:22 PM, Mikael Vidstedt <[hidden email]> wrote:


Looks good.

Cheers,
Mikael - not a Reviewer

On Nov 2, 2018, at 4:05 PM, JC Beyler <[hidden email]> wrote:

Hi all,

Could I get a review for a trivial typo fix:


Reply | Threaded
Open this post in threaded view
|

Re: RFR (XS) 8213246: Fix typo in vmTestbase failuire to failure

JC Beyler
Thanks all, pushed :)
Jc

On Fri, Nov 2, 2018 at 4:25 PM Igor Ignatyev <[hidden email]> wrote:
LGTM

-- Igor - a Reviewer

On Nov 2, 2018, at 4:22 PM, Mikael Vidstedt <[hidden email]> wrote:


Looks good.

Cheers,
Mikael - not a Reviewer

On Nov 2, 2018, at 4:05 PM, JC Beyler <[hidden email]> wrote:

Hi all,

Could I get a review for a trivial typo fix:




--

Thanks,
Jc