Code Review for jdk-stand1

Prepared by:smarks on Thu Feb 24 21:45:45 PST 2011
Workspace:/home/smarks/src/jdk-stand1
Compare against version:qparent
Summary of changes: 613 lines changed: 91 ins; 126 del; 396 mod; 3850 unchg
Patch of changes: jdk-stand1.patch
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/classes/java/util/jar/JarFile.java

rev 3516 : 7021582: convert jar/zip code and tests to use try-with-resources
Reviewed-by: alanb, dholmes, sherman
7 lines changed: 0 ins; 0 del; 7 mod; 729 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/util/jar/JarEntry/GetMethodsReturnClones.java

rev 3516 : 7021582: convert jar/zip code and tests to use try-with-resources
Reviewed-by: alanb, dholmes, sherman
15 lines changed: 1 ins; 2 del; 12 mod; 65 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/util/jar/JarFile/ScanSignedJar.java

rev 3516 : 7021582: convert jar/zip code and tests to use try-with-resources
Reviewed-by: alanb, dholmes, sherman
25 lines changed: 10 ins; 10 del; 5 mod; 51 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/util/zip/Available.java

rev 3516 : 7021582: convert jar/zip code and tests to use try-with-resources
Reviewed-by: alanb, dholmes, sherman
26 lines changed: 8 ins; 4 del; 14 mod; 68 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/util/zip/FileBuilder.java

rev 3516 : 7021582: convert jar/zip code and tests to use try-with-resources
Reviewed-by: alanb, dholmes, sherman
18 lines changed: 0 ins; 1 del; 17 mod; 59 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/util/zip/GZIP/Accordion.java

rev 3516 : 7021582: convert jar/zip code and tests to use try-with-resources
Reviewed-by: alanb, dholmes, sherman
6 lines changed: 0 ins; 4 del; 2 mod; 88 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/util/zip/GZIP/GZIPInputStreamRead.java

rev 3516 : 7021582: convert jar/zip code and tests to use try-with-resources
Reviewed-by: alanb, dholmes, sherman
23 lines changed: 1 ins; 1 del; 21 mod; 78 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/util/zip/InflateIn_DeflateOut.java

rev 3516 : 7021582: convert jar/zip code and tests to use try-with-resources
Reviewed-by: alanb, dholmes, sherman
4 lines changed: 1 ins; 1 del; 2 mod; 283 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/util/zip/InfoZip.java

rev 3516 : 7021582: convert jar/zip code and tests to use try-with-resources
Reviewed-by: alanb, dholmes, sherman
25 lines changed: 1 ins; 5 del; 19 mod; 104 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/util/zip/LargeZip.java

rev 3516 : 7021582: convert jar/zip code and tests to use try-with-resources
Reviewed-by: alanb, dholmes, sherman
47 lines changed: 12 ins; 8 del; 27 mod; 158 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/util/zip/TestEmptyZip.java

rev 3516 : 7021582: convert jar/zip code and tests to use try-with-resources
Reviewed-by: alanb, dholmes, sherman
28 lines changed: 0 ins; 18 del; 10 mod; 131 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/util/zip/ZipCoding.java

rev 3516 : 7021582: convert jar/zip code and tests to use try-with-resources
Reviewed-by: alanb, dholmes, sherman
43 lines changed: 2 ins; 3 del; 38 mod; 92 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/util/zip/ZipFile/Assortment.java

rev 3516 : 7021582: convert jar/zip code and tests to use try-with-resources
Reviewed-by: alanb, dholmes, sherman
7 lines changed: 0 ins; 1 del; 6 mod; 239 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/util/zip/ZipFile/Comment.java

rev 3516 : 7021582: convert jar/zip code and tests to use try-with-resources
Reviewed-by: alanb, dholmes, sherman
28 lines changed: 2 ins; 3 del; 23 mod; 88 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/util/zip/ZipFile/CopyJar.java

rev 3516 : 7021582: convert jar/zip code and tests to use try-with-resources
Reviewed-by: alanb, dholmes, sherman
14 lines changed: 2 ins; 2 del; 10 mod; 36 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/util/zip/ZipFile/CorruptedZipFiles.java

rev 3516 : 7021582: convert jar/zip code and tests to use try-with-resources
Reviewed-by: alanb, dholmes, sherman
18 lines changed: 1 ins; 7 del; 10 mod; 225 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/util/zip/ZipFile/DeleteTempJar.java

rev 3516 : 7021582: convert jar/zip code and tests to use try-with-resources
Reviewed-by: alanb, dholmes, sherman
25 lines changed: 5 ins; 5 del; 15 mod; 90 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/util/zip/ZipFile/EnumAfterClose.java

rev 3516 : 7021582: convert jar/zip code and tests to use try-with-resources
Reviewed-by: alanb, dholmes, sherman
6 lines changed: 2 ins; 0 del; 4 mod; 43 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/util/zip/ZipFile/GetDirEntry.java

rev 3516 : 7021582: convert jar/zip code and tests to use try-with-resources
Reviewed-by: alanb, dholmes, sherman
7 lines changed: 1 ins; 1 del; 5 mod; 37 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/util/zip/ZipFile/LargeZipFile.java

rev 3516 : 7021582: convert jar/zip code and tests to use try-with-resources
Reviewed-by: alanb, dholmes, sherman
46 lines changed: 7 ins; 8 del; 31 mod; 121 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/util/zip/ZipFile/ManyEntries.java

rev 3516 : 7021582: convert jar/zip code and tests to use try-with-resources
Reviewed-by: alanb, dholmes, sherman
13 lines changed: 0 ins; 7 del; 6 mod; 102 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/util/zip/ZipFile/ManyZipFiles.java

rev 3516 : 7021582: convert jar/zip code and tests to use try-with-resources
Reviewed-by: alanb, dholmes, sherman
15 lines changed: 1 ins; 0 del; 14 mod; 124 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/util/zip/ZipFile/ReadAfterClose.java

rev 3516 : 7021582: convert jar/zip code and tests to use try-with-resources
Reviewed-by: alanb, dholmes, sherman
7 lines changed: 3 ins; 0 del; 4 mod; 44 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/util/zip/ZipFile/ReadZip.java

rev 3516 : 7021582: convert jar/zip code and tests to use try-with-resources
Reviewed-by: alanb, dholmes, sherman
65 lines changed: 17 ins; 23 del; 25 mod; 69 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/util/zip/ZipFile/ShortRead.java

rev 3516 : 7021582: convert jar/zip code and tests to use try-with-resources
Reviewed-by: alanb, dholmes, sherman
23 lines changed: 3 ins; 1 del; 19 mod; 44 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/util/zip/zip.java

rev 3516 : 7021582: convert jar/zip code and tests to use try-with-resources
Reviewed-by: alanb, dholmes, sherman
72 lines changed: 11 ins; 11 del; 50 mod; 682 unchg

This code review page was prepared using /home/smarks/bin/webrev (vers 23.18-hg).