archive/jdk
1
0
Fork 0
mirror of https://github.com/openjdk/jdk.git synced 2025-08-27 06:45:07 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

8259699: Reduce char[] copying in URLEncoder.encode(String, Charset)

Browse source 
Reviewed-by: attila, redestad, chegar
This commit is contained in:
Sergey Tsypanov 2021-01-14 14:18:12 +00:00 • committed by Claes Redestad
parent ff3e6e46cd
commit c822eda1d8
1 changed files with 9 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

18
src/java.base/share/classes/java/net/URLEncoder.java
View file

@ -76,7 +76,7 @@ import sun.security.action.GetPropertyAction;
public class URLEncoder {
static BitSet dontNeedEncoding;
static final int caseDiff = ('a' - 'A');
static String dfltEncName = null;
static String dfltEncName;
static {
@ -171,7 +171,7 @@ public class URLEncoder {
* Translates a string into {@code application/x-www-form-urlencoded}
* format using a specific encoding scheme.
* <p>
* This method behaves the same as {@linkplain encode(String s, Charset charset)}
* This method behaves the same as {@linkplain #encode(String s, Charset charset)}
* except that it will {@linkplain java.nio.charset.Charset#forName look up the charset}
* using the given encoding name.
*
@ -225,7 +225,7 @@ public class URLEncoder {
CharArrayWriter charArrayWriter = new CharArrayWriter();
for (int i = 0; i < s.length();) {
int c = (int) s.charAt(i);
int c = s.charAt(i);
//System.out.println("Examining character: " + c);
if (dontNeedEncoding.get(c)) {
if (c == ' ') {
@ -253,7 +253,7 @@ public class URLEncoder {
+ " is high surrogate");
*/
if ( (i+1) < s.length()) {
int d = (int) s.charAt(i+1);
int d = s.charAt(i+1);
/*
System.out.println("\tExamining "
+ Integer.toHexString(d));
@ -270,21 +270,21 @@ public class URLEncoder {
}
}
i++;
} while (i < s.length() && !dontNeedEncoding.get((c = (int) s.charAt(i))));
} while (i < s.length() && !dontNeedEncoding.get((c = s.charAt(i))));
charArrayWriter.flush();
String str = new String(charArrayWriter.toCharArray());
String str = charArrayWriter.toString();
byte[] ba = str.getBytes(charset);
for (int j = 0; j < ba.length; j++) {
for (byte b : ba) {
out.append('%');
char ch = Character.forDigit((ba[j] >> 4) & 0xF, 16);
char ch = Character.forDigit((b >> 4) & 0xF, 16);
// converting to use uppercase letter as part of
// the hex value if ch is a letter.
if (Character.isLetter(ch)) {
ch -= caseDiff;
}
out.append(ch);
ch = Character.forDigit(ba[j] & 0xF, 16);
ch = Character.forDigit(b & 0xF, 16);
if (Character.isLetter(ch)) {
ch -= caseDiff;
}