<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    Thanks Michael, <br>
    <br>
    When it comes to ALL CAPs or SturdlyCaps, it's okay if those strings
    are not translated. <br>
    <br>
    Here is an updated webrev. I removed strings that were no longer in
    use and made some changes to the make-files.<br>
    <br>
    <a class="moz-txt-link-freetext"
      href="http://cr.openjdk.java.net/%7Eegahlin/7017818_4/">http://cr.openjdk.java.net/~egahlin/7017818_4/</a>
    <br>
    <br>
    /E<br>
    <br>
    Michael Fang skrev 2012-05-11 18:16:
    <blockquote cite="mid:4FAD3B63.8070501@oracle.com" type="cite">
      <meta content="text/html; charset=ISO-8859-1"
        http-equiv="Content-Type">
      Thanks so much Erik for the fix! The messages.properties file look
      good. I only reviewed the English <font color="green"><b>messages.properties</b></font>
      file. WPTG will re-translate the ja and zh_CN files. They will
      attempt to leverage existing translation from <font color="red"><b>JConsoleResources_xx.java</b></font>.<br>
      <br>
      I have a comments about another translatability rule that WPTG
      follows:<br>
      <br>
      By Oracle software development guideline, ALL CAPs or StudlyCaps
      strings will not be translated. Examples are:<br>
      <pre>  19 ACTION_CAPITALIZED=ACTION
  20 ACTION_INFO_CAPITALIZED=ACTION_INFO
</pre>
      <pre> 149 MBEAN_INFO=MBeanInfo
 150 MBEAN_NOTIFICATION_INFO=MBeanNotificationInfo
 151 MBEAN_OPERATION_INFO=MBeanOperationInfo
</pre>
      <pre> 206 OBJECT_NAME=ObjectName
</pre>
      <pre> 231 R_FORWARD_SLASH_W_CAPITALIZED=R/W

 241 SEQ_NUM=SeqNum
</pre>
      <pre> 249 SUMMARY_TAB_HEADER_DATE_TIME_FORMAT=FULL,FULL

 273 UNKNOWN_CAPITALIZED=UNKNOWN

</pre>
      If any of the above such as ACTION or UNKNOWN needs to be
      translated, they should not be ALL CAPS. The translators will not
      translate those lines by default.<br>
      <br>
      The translators have the ability to see comments (or special
      requests) if a comment line is inserted immediately prior to any
      of the resources (each resource string and one comment line
      immediately before it is stored in translation memory). If you
      must leave ACTION or UNKNOWN in CAPS, you can try to insert
      comments.However, WPTG does not guarantee they will be followed.<br>
      <br>
      thanks,<br>
      <br>
      -michael<br>
      <br>
      On 12&#24180;05&#26376;11&#26085; 07:36 &#19978;&#21320;, Erik Gahlin wrote:
      <blockquote cite="mid:4FAD23E4.90501@oracle.com" type="cite">Could
        you please review? I also need a sponsor. <br>
        <br>
        <a moz-do-not-send="true" class="moz-txt-link-freetext"
          href="http://cr.openjdk.java.net/%7Eegahlin/7156518/1_0/">http://cr.openjdk.java.net/~egahlin/7156518/1_0/</a>
        <br>
        <br>
        <a moz-do-not-send="true" class="moz-txt-link-freetext"
          href="http://monaco.us.oracle.com/detail.jsf?cr=7017818">http://monaco.us.oracle.com/detail.jsf?cr=7017818</a>
        <br>
        <br>
        The patch is for JDK8, but it needs to ported to 7u6 before
        5/16. <br>
        <br>
        Thanks! <br>
        <br>
        Erik <br>
        <br>
        Changes: <br>
        <br>
        - Moved localization messages to property files, one message per
        line, as needed. <br>
        - Added '&amp;' to messages so mnemonics could be identified. <br>
        - Introduced Message class with static fields corresponding to
        the keys in the property files. <br>
        - Added map for looking up mnemonics. <br>
        <br>
        Testing: <br>
        <br>
        - Verified programmatically that all the messages and mnemonics
        are "compatible" with the previous mechanism, for Chinese,
        Japanese and the default locale. <br>
        - The intention is to run through the GUI to confirm that
        everything looks ok, but I'm waiting for a build. <br>
        <br>
        Other: <br>
        <br>
        - Fixed a typo in the MemoryPoolStat class, the method
        getAfterGcUsage returned this.beforeGCUsage instead of
        this.afterGcUsage <br>
        - When going through all the code I did some minor clean up that
        should not impact the program flow: <br>
        &nbsp; * removed unused imports. <br>
        &nbsp; * inlined temporary variables holding messages. <br>
        &nbsp; * removed private member variables that were not accessed. <br>
        &nbsp; * removed private methods that were not referenced. <br>
        &nbsp; * removed local variables that were not used. <br>
        &nbsp; * added generic types where needed. <br>
        &nbsp; * static methods are now called statically. <br>
        &nbsp; * added suppress serialization warning where it was needed. <br>
        &nbsp; * removed suppress serialization annotation where it was not
        needed. <br>
        - In the Message class, the comment "remove? not found in code"
        will be removed once I know those message are not needed for
        7u6. <br>
        <br>
      </blockquote>
    </blockquote>
    <br>
    <br>
  </body>
</html>