/hg/icedtea6: Backport of: 6792400: Avoid loading of Normalizer ...

[ptisnovs at icedtea.classpath.org]

changeset 86fa5b8eb7a9 in /hg/icedtea6
details: http://icedtea.classpath.org/hg/icedtea6?cmd=changeset;node=86fa5b8eb7a9
author: ptisnovs
date: Tue Mar 27 15:30:53 2012 +0200

	Backport of: 6792400: Avoid loading of Normalizer resources for simple uses

	* patches/openjdk/6792400-Avoid_loading_Normalizer_resources.patch:
	  Added new patch
	* NEWS: Mention backport.
	* Makefile.am: Updated


diffstat:

 ChangeLog                                                        |   8 +
 Makefile.am                                                      |   3 +-
 NEWS                                                             |   1 +
 patches/openjdk/6792400-Avoid_loading_Normalizer_resources.patch |  49 ++++++++++
 4 files changed, 60 insertions(+), 1 deletions(-)

diffs (93 lines):

diff -r e5b8981cf2ab -r 86fa5b8eb7a9 ChangeLog
--- a/ChangeLog	Mon Mar 26 23:45:05 2012 +0200
+++ b/ChangeLog	Tue Mar 27 15:30:53 2012 +0200
@@ -1,3 +1,11 @@
+2012-03-27  Pavel Tisnovsky  <ptisnovs at redhat.com>
+
+	* patches/openjdk/6792400-Avoid_loading_Normalizer_resources.patch:
+	Backport of: 6792400: Avoid loading of Normalizer resources
+	for simple uses
+	* NEWS: Mention backport.
+	* Makefile.am: Updated
+
 2012-03-21  Pavel Tisnovsky  <ptisnovs at redhat.com>
 
 	* patches/apache-xml-internal-fix-bug-38655.patch:
diff -r e5b8981cf2ab -r 86fa5b8eb7a9 Makefile.am
--- a/Makefile.am	Mon Mar 26 23:45:05 2012 +0200
+++ b/Makefile.am	Tue Mar 27 15:30:53 2012 +0200
@@ -424,7 +424,8 @@
 	patches/openjdk/6883983-JarVerifier_removed_dependency_sun_security_pkcs.patch \
 	patches/openjdk/4465490-Suspicious_double-check_locking_idiom.patch \
 	patches/idresolver_fix.patch \
-	patches/apache-xml-internal-fix-bug-38655.patch
+	patches/apache-xml-internal-fix-bug-38655.patch \
+	patches/openjdk/6792400-Avoid_loading_Normalizer_resources.patch
 
 if WITH_RHINO
 ICEDTEA_PATCHES += \
diff -r e5b8981cf2ab -r 86fa5b8eb7a9 NEWS
--- a/NEWS	Mon Mar 26 23:45:05 2012 +0200
+++ b/NEWS	Tue Mar 27 15:30:53 2012 +0200
@@ -22,6 +22,7 @@
   - S6761072: new krb5 tests fail on multiple platforms
   - S6883983: JarVerifier dependency on sun.security.pkcs should be removed
   - S4465490: Suspicious about double-check locking idiom being used in the code
+  - S6792400: Avoid loading of Normalizer resources for simple uses
 
 New in release 1.10.6 (2012-02-14):
 
diff -r e5b8981cf2ab -r 86fa5b8eb7a9 patches/openjdk/6792400-Avoid_loading_Normalizer_resources.patch
--- /dev/null	Thu Jan 01 00:00:00 1970 +0000
+++ b/patches/openjdk/6792400-Avoid_loading_Normalizer_resources.patch	Tue Mar 27 15:30:53 2012 +0200
@@ -0,0 +1,50 @@
+# HG changeset patch
+# User peytoia
+# Date 1245934539 -32400
+# Node ID e0707baa159364ff923bf901eab1b6c83d4cf092
+# Parent  4d54d6e7bcefd16bcc9a26b93937359f031389ca
+6792400: Avoid loading of Normalizer resources for simple uses
+Reviewed-by: okutsu
+
+diff -r 4d54d6e7bcef -r e0707baa1593 src/share/classes/sun/text/normalizer/NormalizerBase.java
+--- openjdk/jdk/src/share/classes/sun/text/normalizer/NormalizerBase.java	Thu Jun 25 02:42:26 2009 -0700
++++ openjdk/jdk/src/share/classes/sun/text/normalizer/NormalizerBase.java	Thu Jun 25 21:55:39 2009 +0900
+@@ -1598,15 +1598,34 @@
+      * @param options   the optional features to be enabled.
+      */
+     public static String normalize(String str, Normalizer.Form form, int options) {
++        int len = str.length();
++        boolean asciiOnly = true;
++        if (len < 80) {
++            for (int i = 0; i < len; i++) {
++                if (str.charAt(i) > 127) {
++                    asciiOnly = false;
++                    break;
++                }
++            }
++        } else {
++            char[] a = str.toCharArray();
++            for (int i = 0; i < len; i++) {
++                if (a[i] > 127) {
++                    asciiOnly = false;
++                    break;
++                }
++            }
++        }
++
+         switch (form) {
+         case NFC :
+-            return NFC.normalize(str, options);
++            return asciiOnly ? str : NFC.normalize(str, options);
+         case NFD :
+-            return NFD.normalize(str, options);
++            return asciiOnly ? str : NFD.normalize(str, options);
+         case NFKC :
+-            return NFKC.normalize(str, options);
++            return asciiOnly ? str : NFKC.normalize(str, options);
+         case NFKD :
+-            return NFKD.normalize(str, options);
++            return asciiOnly ? str : NFKD.normalize(str, options);
+         }
+ 
+         throw new IllegalArgumentException("Unexpected normalization form: " +