[8u-dev] RFA (XS): JDK-8211394: CHECK_ must be used in the rhs of an assignment statement within a block

[Hohensee, Paul]

Done and pushed. Thanks!

On 10/18/18, 8:41 PM, "David Buck" <david.buck at oracle.com> wrote:

    Hi Paul!
    
    Please add an appropriate noreg label [0] to the JBS bug report.
    
    Once that is done please consider this approved for push to 8u-dev.
    
    Cheers,
    -Buck
    
    [0] https://openjdk.java.net/guide/changePlanning.html#noreg
    
    On 2018/10/19 9:29, David Holmes wrote:
    > On 19/10/2018 2:44 AM, Hohensee, Paul wrote:
    >> Hi David,
    >>
    >> Did the jprt job go ok? If so, Rob or Sean, would you please approve 
    >> the backport?
    > 
    > Yes - sorry. Didn't expect it to hold this up as it's so trivial.
    > 
    > David
    > 
    >> Thanks,
    >>
    >> Paul
    >>
    >> On 10/16/18, 7:06 PM, "David Holmes" <david.holmes at oracle.com> wrote:
    >>
    >>      On 17/10/2018 8:23 AM, Hohensee, Paul wrote:
    >>      > Please approve a trivial backport of JDK-8211394 to 8u.
    >>      >
    >>      > JBS: https://bugs.openjdk.java.net/browse/JDK-8211394
    >>      >
    >>      > Webrev: http://cr.openjdk.java.net/~phh/8211394/webrev.00/
    >>      > <http://cr.openjdk.java.net/%7Ephh/8211394/webrev.00/>
    >>      >
    >>      > Review thread:
    >>      > 
    >> http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2018-October/030376.html 
    >>
    >>      >
    >>      > The patch applies cleanly net of line numbers, copyright notice
    >>      > dates, and code that exists in JDK9 but not in JDK8. The result 
    >> is a one
    >>      > line change and a new comment.
    >>      Looks fine.
    >>      > It builds and runs jdk8 hotspot jtreg test on a linux x64 host. If
    >>      > needed, would someone please run it through jprt?
    >>      It doesn't really warrant it but I fired it off anyway.
    >>      Thanks,
    >>      David
    >>      > Thanks,
    >>      >
    >>      > Paul
    >>      >
    >>