[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-serviceability-dev
Subject:    Re: RFR: 8080928 Uninitialised variable in hotspot/src/share/vm/prims/jvmtiEnvBase.cpp
From:       "serguei.spitsyn () oracle ! com" <serguei ! spitsyn () oracle ! com>
Date:       2015-05-28 10:54:02
Message-ID: 5566F3CA.10401 () oracle ! com
[Download RAW message or body]

Looks good.

Thanks,
Serguei

On 5/28/15 3:38 AM, Staffan Larsen wrote:
> Please review this simple fix for an uninitialized variable.
>
> webrev: 
> http://cr.openjdk.java.net/~sla/8080928/webrev.00/src/share/vm/prims/jvmtiEnvBase.cpp.sdiff.html 
> <http://cr.openjdk.java.net/%7Esla/8080928/webrev.00/src/share/vm/prims/jvmtiEnvBase.cpp.sdiff.html> 
>
>
> Thanks,
> /Staffan


[Attachment #3 (text/html)]

<html>
  <head>
    <meta content="text/html; charset=windows-1252"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix">Looks good.<br>
      <br>
      Thanks,<br>
      Serguei<br>
      <br>
      On 5/28/15 3:38 AM, Staffan Larsen wrote:<br>
    </div>
    <blockquote
      cite="mid:FBC3A9E1-5AC2-4D17-B5E3-5756715C8C36@oracle.com"
      type="cite">
      <meta http-equiv="Content-Type" content="text/html;
        charset=windows-1252">
      <div class="">Please review this simple fix for an uninitialized
        variable.</div>
      <div class=""><br class="">
      </div>
      webrev: <a moz-do-not-send="true"
href="http://cr.openjdk.java.net/%7Esla/8080928/webrev.00/src/share/vm/prims/jvmtiEnvBase.cpp.sdiff.html"
                
        class="">http://cr.openjdk.java.net/~sla/8080928/webrev.00/src/share/vm/prims/jvmtiEnvBase.cpp.sdiff.html</a>
  <div class=""><br class="">
      </div>
      <div class="">
        <div class="">Thanks,</div>
        <div class="">/Staffan</div>
      </div>
    </blockquote>
    <br>
  </body>
</html>



[prev in list] [next in list] [prev in thread] [next in thread]

Configure | About | News | Add a list | Sponsored by KoreLogic