Re: RFR: 8235788: Changeset for JDK-8235252 pushed with wrong bug ID

2019-12-11 Thread Alexey Semenyuk 

Looks good.

- Alexey

On 12/11/2019 2:57 PM, Phil Race wrote:

+1

-phil

On 12/11/19 11:43 AM, Andy Herrick wrote:

sorry - fix is at:

[3] http://cr.openjdk.java.net/~herrick/8235788/webrev.01/

/Andy

On 12/11/2019 2:42 PM, Andy Herrick wrote:

Please review fix [1] for issue [2]

This is backing out the fix to JDK-8235252 so I can re-push it with 
the right comment


[1] 8235788  
Changeset for JDK-8235252 pushed with wrong bug ID


[2] https://bugs.openjdk.java.net/browse/JDK-8235788

/Andy

Re: RFR: 8235788: Changeset for JDK-8235252 pushed with wrong bug ID

2019-12-11 Thread Phil Race 

+1

-phil

On 12/11/19 11:43 AM, Andy Herrick wrote:

sorry - fix is at:

[3] http://cr.openjdk.java.net/~herrick/8235788/webrev.01/

/Andy

On 12/11/2019 2:42 PM, Andy Herrick wrote:

Please review fix [1] for issue [2]

This is backing out the fix to JDK-8235252 so I can re-push it with 
the right comment


[1] 8235788  
Changeset for JDK-8235252 pushed with wrong bug ID


[2] https://bugs.openjdk.java.net/browse/JDK-8235788

/Andy

Re: RFR: 8235788: Changeset for JDK-8235252 pushed with wrong bug ID

2019-12-11 Thread Andy Herrick 

sorry - fix is at:

[3] http://cr.openjdk.java.net/~herrick/8235788/webrev.01/

/Andy

On 12/11/2019 2:42 PM, Andy Herrick wrote:

Please review fix [1] for issue [2]

This is backing out the fix to JDK-8235252 so I can re-push it with 
the right comment


[1] 8235788  
Changeset for JDK-8235252 pushed with wrong bug ID


[2] https://bugs.openjdk.java.net/browse/JDK-8235788

/Andy