<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    Hi Daven,<br>
    <br>
    In FormatData_sl.java, the data are<br>
    <br>
    &nbsp;134&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; "EEEE, dd. MMMM yyyy", // full date pattern<br>
    &nbsp;135&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; "EEEE, dd. MMMM yyyy", // long date pattern<br>
    <br>
    I think they should be changed to <br>
    <br>
    &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; "EEEE, dd. MMMM y", // full date pattern<br>
    &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; "dd. MMMM y", // long date pattern<br>
    <br>
    the same as what in test file LocaleData.<br>
    <br>
    The change in LocaleData and LocaleDataTest.java are good.<br>
    <br>
    thanks for fixing the CR.<br>
    Yong<br>
    <br>
    On 2012/5/24 13:51, Deven You wrote:
    <blockquote cite="mid:4FBDCC5B.1050501@linux.vnet.ibm.com"
      type="cite">
      <meta content="text/html; charset=ISO-8859-1"
        http-equiv="Content-Type">
      Hi Jeffrey,<br>
      <br>
      I have updated the webrev[1] according to your comments. Please
      review it.<br>
      <br>
      [1]
      <meta http-equiv="content-type" content="text/html;
        charset=ISO-8859-1">
      <a moz-do-not-send="true"
        href="http://cr.openjdk.java.net/%7Elittlee/OJDK-126/webrev.00/">http://cr.openjdk.java.net/~littlee/OJDK-126/webrev.00/</a>
      <br>
      <br>
      Thanks a lot!<br>
      <br>
      On 05/21/2012 05:27 PM, Yong Jeffrey Huang wrote:
      <blockquote cite="mid:4FBA0A76.40209@oracle.com" type="cite">
        <meta content="text/html; charset=ISO-8859-1"
          http-equiv="Content-Type">
        Hi Daven,<br>
        <br>
        We use the format data in CLDR as the standard. Attached is the
        sl.xml in CLDR. In your fix, it's better to be<br>
        &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; "EEEE, dd. MMMM y", // full date pattern<br>
        &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; "dd. MMMM y", // long date pattern<br>
        <br>
        Blow is the format defined in CLDR:<br>
        <br>
        - &lt;dateFormatLength type="full"&gt;<br>
        - &lt;dateFormat type="standard"&gt;<br>
        &nbsp; &lt;pattern type="standard"&gt;EEEE, dd. MMMM
        y&lt;/pattern&gt; <br>
        &nbsp; &lt;/dateFormat&gt;<br>
        &nbsp; &lt;/dateFormatLength&gt;<br>
        - &lt;dateFormatLength type="long"&gt;<br>
        - &lt;dateFormat type="standard"&gt;<br>
        &nbsp; &lt;pattern type="standard"&gt;dd. MMMM y&lt;/pattern&gt; <br>
        &nbsp; &lt;/dateFormat&gt;<br>
        &nbsp; &lt;/dateFormatLength&gt; <br>
        <br>
        <big>For test case:</big><br>
        <br>
        To fix format data of one locale, you must modify corresponding
        regression test:<br>
        <br>
        jdk/test/sun/text/resources/LocaleData:<br>
        In the end of the file,&nbsp; add<br>
        #bug xxxxxx<br>
        FormatData/sl/DateTimePatterns/4=EEEE, dd. MMMM y<br>
        FormatData/sl/DateTimePatterns/5=dd. MMMM y<br>
        <br>
        jdk/test/sun/text/resources/LocaleDataTest.java:<br>
        In the end of comments @bug, add the bug number.<br>
        <br>
        thanks,<br>
        Yong<br>
        <br>
        <br>
        On 2012/5/21 15:41, Deven You wrote:
        <blockquote cite="mid:4FB9F1AF.5000106@linux.vnet.ibm.com"
          type="cite">Hi All, <br>
          <br>
          I think this patch is simple and straight forward, is anyone
          interested in taking a look? <br>
          <br>
          Thanks a lot! <br>
          <br>
          On 03/05/2012 01:52 PM, Deven wrote: <br>
          <blockquote type="cite">Hi i18n-devs, <br>
            <br>
            I find there is a problem in Slovenian Locale. In Slovenian
            Locale, days in a date must with a dot '.', like Oct 23rd
            2011 will be 23. oktober 2011. Without the '.', 23 oktober
            2011 basically means Twenty-three of October, instead of the
            Twenty-third of October. <br>
            <br>
            Current oracle 6/7/8 all miss the dot '.', so this patch [1]
            will add dot after the days to correct the date expression
            in Slovenian Locale. <br>
            <br>
            The attached test case [2] shows the the diffrence
            with/without this patch. The output of this test case are: <br>
            <br>
            without this patch: <br>
            Ponedeljek, 5 marec 2012 <br>
            <br>
            with this patch: <br>
            Ponedeljek, 5. marec 2012 <br>
            <br>
            This test case may not be very useful, so I hope anyone can
            give me some suggestions about how to refine the test case?
            <br>
            <br>
            [1] <a moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Eyoudwei/slovenian_formatData/webrev.00/">http://cr.openjdk.java.net/~youdwei/slovenian_formatData/webrev.00/</a>
            <a moz-do-not-send="true" class="moz-txt-link-rfc2396E"
href="http://cr.openjdk.java.net/%7Eyoudwei/slovenian_formatData/webrev.00/">&lt;http://cr.openjdk.java.net/%7Eyoudwei/slovenian_formatData/webrev.00/&gt;</a>
            <br>
            <br>
            [2] SlovenianFormatDataTest.java <br>
            <br>
            Thanks a lot! <br>
            <br>
          </blockquote>
          <br>
          <br>
        </blockquote>
        <br>
      </blockquote>
      <br>
      <br>
      <pre class="moz-signature" cols="72">-- 
Best Regards,

Deven</pre>
    </blockquote>
    <br>
  </body>
</html>