8035054: JarFacade.c should not include ctype.h
authormikael
Mon, 24 Feb 2014 11:24:43 -0800
changeset 23014 af8145529dd4
parent 23013 1cf3dddf9ff7
child 23015 73b21ab36615
8035054: JarFacade.c should not include ctype.h Reviewed-by: alanb
jdk/src/share/instrument/JarFacade.c
--- a/jdk/src/share/instrument/JarFacade.c	Tue Feb 25 11:51:25 2014 +0400
+++ b/jdk/src/share/instrument/JarFacade.c	Mon Feb 24 11:24:43 2014 -0800
@@ -23,17 +23,8 @@
  * questions.
  */
 
-#ifdef _WIN32
-/*
- * Win* needs this include. However, Linux and Solaris do not.
- * Having this include on Solaris SPARC breaks having non US-ASCII
- * characters in the value of the Premain-Class attribute.
- */
-#include <ctype.h>
-#endif /* _WIN32 */
 #include <string.h>
 #include <stdlib.h>
-#include <ctype.h>
 
 #include "jni.h"
 #include "manifest_info.h"
@@ -59,13 +50,13 @@
             size_t value_len;
 
             /* skip any leading white space */
-            while (isspace(*begin)) {
+            while (*begin == ' ') {
                 begin++;
             }
 
             /* skip any trailing white space */
             end = &begin[strlen(begin)];
-            while (end > begin && isspace(end[-1])) {
+            while (end > begin && end[-1] == ' ') {
                 end--;
             }