Xueming,<br><br>Congratulations for finding the incompatible change!<br><br>Yes, if you are running in a turkish locale (unlikely)<br>and if your jar file has the index entry in lower case,<br>(unlikely - hopefully INDEX.LIST files are all machine generated).<br>
then it will not be found in the old implementation, <br>and will be found in the new one.<br>Can we consider this a simple bug fix?  Please?<br>Even if all of the above occurs, the change in behavior may be benign -<br>the resulting jar file will contain both upper and lower case index files.<br>
Perhaps I should add a test, but it would be painful to write one.<br><br>It was never a good idea to make these entry names case-insensitive.<br><br>Martin<br><br><div class="gmail_quote">On Thu, Jun 25, 2009 at 09:12, Xueming Shen <span dir="ltr"><<a href="mailto:Xueming.Shen@sun.com">Xueming.Shen@sun.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><br>
do we have a Turkish i "regression" for the index name comparing by assuming ASCII only?<br>
yes, the original impl is not consistent already when dealing with manifest and index , one use<br>
toUpperCase(Locale.ENGLISH), one does not.<br>
<br>
<br>
Martin Buchholz wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div><div></div><div class="h5">
Hi jar team,<br>
<br>
I have a bunch of minor improvements to<br>
 src/share/classes/sun/tools/jar/Main.java<br>
<br>
Toby and Xueming, please review.<br>
<br>
Warning: the index code has not been maintained for many years.<br>
<br>
Xueming, please file a bug.<br>
<br>
Synopsis: Miscellaneous improvements to "jar".<br>
Description:<br>
- Use standard jdk coding style for javadoc<br>
- Don't create a temp file for jar index in STORED mode.<br>
- Don't use synchronized collections.<br>
- Fix javac warnings.<br>
- Don't define new names for things like INDEX_NAME;<br>
  use static import instead.<br>
- more efficiently compare special file names in update mode.<br>
  Update mode should be measurably faster.<br>
- make CRC32OutputStream a nested class.<br>
  refactor crc32.reset and updating entry into CRC32OutputStream.<br>
- Fix apparently benign bug updating n in CRC32OutputStream.write(byte[], int, int)<br>
<br>
Evaluation: Yep.<br>
<br>
</div></div><div class="im"><a href="http://cr.openjdk.java.net/%7Emartin/jar-misc/" target="_blank">http://cr.openjdk.java.net/~martin/jar-misc/</a> <<a href="http://cr.openjdk.java.net/%7Emartin/jar-misc/" target="_blank">http://cr.openjdk.java.net/%7Emartin/jar-misc/</a>><br>

<br>
Thanks,<br>
<br>
Martin<br>
</div></blockquote>
<br>
</blockquote></div><br>