Skip to content

Commit 8c70d62

Browse files
committed
TruffleStrings: keep overloaded versions of nodes with new ErrorHandling parameter.
1 parent a95c0e6 commit 8c70d62

File tree

4 files changed

+9
-103
lines changed

4 files changed

+9
-103
lines changed

truffle/src/com.oracle.truffle.api.strings/snapshot.sigtest

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -98,14 +98,11 @@ meth public final int byteIndexOfStringUncached(com.oracle.truffle.api.strings.T
9898
meth public final int byteIndexToCodePointIndexUncached(int,int,com.oracle.truffle.api.strings.TruffleString$Encoding)
9999
meth public final int byteLength(com.oracle.truffle.api.strings.TruffleString$Encoding)
100100
meth public final int byteLengthOfCodePointUncached(int,com.oracle.truffle.api.strings.TruffleString$Encoding)
101-
anno 0 java.lang.Deprecated(boolean forRemoval=false, java.lang.String since="22.3")
102101
meth public final int byteLengthOfCodePointUncached(int,com.oracle.truffle.api.strings.TruffleString$Encoding,com.oracle.truffle.api.strings.TruffleString$ErrorHandling)
103102
meth public final int charIndexOfAnyCharUTF16Uncached(int,int,char[])
104103
meth public final int codePointAtByteIndexUncached(int,com.oracle.truffle.api.strings.TruffleString$Encoding)
105-
anno 0 java.lang.Deprecated(boolean forRemoval=false, java.lang.String since="22.3")
106104
meth public final int codePointAtByteIndexUncached(int,com.oracle.truffle.api.strings.TruffleString$Encoding,com.oracle.truffle.api.strings.TruffleString$ErrorHandling)
107105
meth public final int codePointAtIndexUncached(int,com.oracle.truffle.api.strings.TruffleString$Encoding)
108-
anno 0 java.lang.Deprecated(boolean forRemoval=false, java.lang.String since="22.3")
109106
meth public final int codePointAtIndexUncached(int,com.oracle.truffle.api.strings.TruffleString$Encoding,com.oracle.truffle.api.strings.TruffleString$ErrorHandling)
110107
meth public final int codePointIndexToByteIndexUncached(int,int,com.oracle.truffle.api.strings.TruffleString$Encoding)
111108
meth public final int codePointLengthUncached(com.oracle.truffle.api.strings.TruffleString$Encoding)
@@ -370,7 +367,6 @@ CLSS public abstract static com.oracle.truffle.api.strings.TruffleString$ByteLen
370367
outer com.oracle.truffle.api.strings.TruffleString
371368
meth public abstract int execute(com.oracle.truffle.api.strings.AbstractTruffleString,int,com.oracle.truffle.api.strings.TruffleString$Encoding,com.oracle.truffle.api.strings.TruffleString$ErrorHandling)
372369
meth public final int execute(com.oracle.truffle.api.strings.AbstractTruffleString,int,com.oracle.truffle.api.strings.TruffleString$Encoding)
373-
anno 0 java.lang.Deprecated(boolean forRemoval=false, java.lang.String since="22.3")
374370
meth public static com.oracle.truffle.api.strings.TruffleString$ByteLengthOfCodePointNode create()
375371
meth public static com.oracle.truffle.api.strings.TruffleString$ByteLengthOfCodePointNode getUncached()
376372
supr com.oracle.truffle.api.nodes.Node
@@ -386,7 +382,6 @@ CLSS public abstract static com.oracle.truffle.api.strings.TruffleString$CodePoi
386382
outer com.oracle.truffle.api.strings.TruffleString
387383
meth public abstract int execute(com.oracle.truffle.api.strings.AbstractTruffleString,int,com.oracle.truffle.api.strings.TruffleString$Encoding,com.oracle.truffle.api.strings.TruffleString$ErrorHandling)
388384
meth public final int execute(com.oracle.truffle.api.strings.AbstractTruffleString,int,com.oracle.truffle.api.strings.TruffleString$Encoding)
389-
anno 0 java.lang.Deprecated(boolean forRemoval=false, java.lang.String since="22.3")
390385
meth public static com.oracle.truffle.api.strings.TruffleString$CodePointAtByteIndexNode create()
391386
meth public static com.oracle.truffle.api.strings.TruffleString$CodePointAtByteIndexNode getUncached()
392387
supr com.oracle.truffle.api.nodes.Node
@@ -395,7 +390,6 @@ CLSS public abstract static com.oracle.truffle.api.strings.TruffleString$CodePoi
395390
outer com.oracle.truffle.api.strings.TruffleString
396391
meth public abstract int execute(com.oracle.truffle.api.strings.AbstractTruffleString,int,com.oracle.truffle.api.strings.TruffleString$Encoding,com.oracle.truffle.api.strings.TruffleString$ErrorHandling)
397392
meth public final int execute(com.oracle.truffle.api.strings.AbstractTruffleString,int,com.oracle.truffle.api.strings.TruffleString$Encoding)
398-
anno 0 java.lang.Deprecated(boolean forRemoval=false, java.lang.String since="22.3")
399393
meth public static com.oracle.truffle.api.strings.TruffleString$CodePointAtIndexNode create()
400394
meth public static com.oracle.truffle.api.strings.TruffleString$CodePointAtIndexNode getUncached()
401395
supr com.oracle.truffle.api.nodes.Node

truffle/src/com.oracle.truffle.api.strings/src/com/oracle/truffle/api/strings/AbstractTruffleString.java

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -599,14 +599,8 @@ public final int readCharUTF16Uncached(int i) {
599599
* Shorthand for calling the uncached version of
600600
* {@link TruffleString.CodePointIndexToByteIndexNode}.
601601
*
602-
* @deprecated since 22.3, use
603-
* {@link #byteLengthOfCodePointUncached(int, Encoding, TruffleString.ErrorHandling)}
604-
* instead.
605-
*
606602
* @since 22.1
607603
*/
608-
@Deprecated(since = "22.3")
609-
@SuppressWarnings("deprecated")
610604
@TruffleBoundary
611605
public final int byteLengthOfCodePointUncached(int byteIndex, TruffleString.Encoding expectedEncoding) {
612606
return TruffleString.ByteLengthOfCodePointNode.getUncached().execute(this, byteIndex, expectedEncoding);
@@ -648,14 +642,8 @@ public final int codePointIndexToByteIndexUncached(int byteOffset, int codepoint
648642
/**
649643
* Shorthand for calling the uncached version of {@link TruffleString.CodePointAtIndexNode}.
650644
*
651-
* @deprecated since 22.3, use
652-
* {@link #codePointAtIndexUncached(int, Encoding, TruffleString.ErrorHandling)}
653-
* instead.
654-
*
655645
* @since 22.1
656646
*/
657-
@Deprecated(since = "22.3")
658-
@SuppressWarnings("deprecated")
659647
@TruffleBoundary
660648
public final int codePointAtIndexUncached(int i, TruffleString.Encoding expectedEncoding) {
661649
return TruffleString.CodePointAtIndexNode.getUncached().execute(this, i, expectedEncoding);
@@ -674,14 +662,8 @@ public final int codePointAtIndexUncached(int i, TruffleString.Encoding expected
674662
/**
675663
* Shorthand for calling the uncached version of {@link TruffleString.CodePointAtByteIndexNode}.
676664
*
677-
* @deprecated since 22.3, use
678-
* {@link #codePointAtByteIndexUncached(int, Encoding, TruffleString.ErrorHandling)}
679-
* instead.
680-
*
681665
* @since 22.1
682666
*/
683-
@Deprecated(since = "22.3")
684-
@SuppressWarnings("deprecated")
685667
@TruffleBoundary
686668
public final int codePointAtByteIndexUncached(int i, TruffleString.Encoding expectedEncoding) {
687669
return TruffleString.CodePointAtByteIndexNode.getUncached().execute(this, i, expectedEncoding);

truffle/src/com.oracle.truffle.api.strings/src/com/oracle/truffle/api/strings/SuppressFBWarnings.java

Lines changed: 0 additions & 60 deletions
This file was deleted.

truffle/src/com.oracle.truffle.api.strings/src/com/oracle/truffle/api/strings/TruffleString.java

Lines changed: 9 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -2950,15 +2950,11 @@ public abstract static class ByteLengthOfCodePointNode extends Node {
29502950
}
29512951

29522952
/**
2953-
* Get the number of bytes occupied by the codepoint starting at {@code byteIndex}.
2954-
*
2955-
* @deprecated since 22.3, use
2956-
* {@link #execute(AbstractTruffleString, int, Encoding, ErrorHandling)}
2957-
* instead.
2953+
* Get the number of bytes occupied by the codepoint starting at {@code byteIndex}, with
2954+
* {@link ErrorHandling#BEST_EFFORT best-effort} error handling.
29582955
*
29592956
* @since 22.1
29602957
*/
2961-
@Deprecated(since = "22.3")
29622958
public final int execute(AbstractTruffleString a, int byteIndex, Encoding expectedEncoding) {
29632959
return execute(a, byteIndex, expectedEncoding, ErrorHandling.BEST_EFFORT);
29642960
}
@@ -3146,14 +3142,11 @@ public abstract static class CodePointAtIndexNode extends Node {
31463142
}
31473143

31483144
/**
3149-
* Decode and return the codepoint at codepoint index {@code i}.
3150-
*
3151-
* @deprecated since 22.3, use
3152-
* {@link #execute(AbstractTruffleString, int, Encoding, ErrorHandling)}
3153-
* instead.
3145+
* Decode and return the codepoint at codepoint index {@code i}, with
3146+
* {@link ErrorHandling#BEST_EFFORT best-effort} error handling.
3147+
*
31543148
* @since 22.1
31553149
*/
3156-
@Deprecated(since = "22.3")
31573150
public final int execute(AbstractTruffleString a, int i, Encoding expectedEncoding) {
31583151
return execute(a, i, expectedEncoding, ErrorHandling.BEST_EFFORT);
31593152
}
@@ -3230,15 +3223,11 @@ public abstract static class CodePointAtByteIndexNode extends Node {
32303223
}
32313224

32323225
/**
3233-
* Decode and return the codepoint at byte index {@code i}.
3234-
*
3235-
* @deprecated since 22.3, use
3236-
* {@link #execute(AbstractTruffleString, int, Encoding, ErrorHandling)}
3237-
* instead.
3226+
* Decode and return the codepoint at byte index {@code i}, with
3227+
* {@link ErrorHandling#BEST_EFFORT best-effort} error handling.
32383228
*
32393229
* @since 22.1
32403230
*/
3241-
@Deprecated(since = "22.3")
32423231
public final int execute(AbstractTruffleString a, int i, Encoding expectedEncoding) {
32433232
return execute(a, i, expectedEncoding, ErrorHandling.BEST_EFFORT);
32443233
}
@@ -4882,6 +4871,7 @@ public static EqualNode getUncached() {
48824871
*
48834872
* @since 22.1
48844873
*/
4874+
@SuppressWarnings("serial")
48854875
public static final class NumberFormatException extends Exception {
48864876

48874877
private static final long serialVersionUID = 0x016db657faff57a2L;
@@ -4945,7 +4935,7 @@ public String getMessage() {
49454935
}
49464936
}
49474937

4948-
@SuppressFBWarnings(value = {"SE_BAD_FIELD"}, justification = "This Exception is not expected to be serialized") private final AbstractTruffleString string;
4938+
private final AbstractTruffleString string;
49494939
private final int regionOffset;
49504940
private final int regionLength;
49514941
private final Reason reason;

0 commit comments

Comments
 (0)