Skip to content

Commit 5d18662

Browse files
committed
[GR-23348] make test_userstring pass
PullRequest: graalpython/1397
2 parents c36ebb3 + 47d222b commit 5d18662

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

graalpython/com.oracle.graal.python.test/src/tests/unittest_tags/test_userstring.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
*graalpython.lib-python.3.test.test_userstring.UserStringTest.test_count
88
*graalpython.lib-python.3.test.test_userstring.UserStringTest.test_encode_default_args
99
*graalpython.lib-python.3.test.test_userstring.UserStringTest.test_encode_explicit_none_args
10+
*graalpython.lib-python.3.test.test_userstring.UserStringTest.test_endswith
1011
*graalpython.lib-python.3.test.test_userstring.UserStringTest.test_expandtabs
1112
*graalpython.lib-python.3.test.test_userstring.UserStringTest.test_extended_getslice
1213
*graalpython.lib-python.3.test.test_userstring.UserStringTest.test_find

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/str/StringBuiltins.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -575,7 +575,7 @@ abstract static class PrefixSuffixBaseNode extends PythonQuaternaryBuiltinNode {
575575
@Specialization
576576
boolean doStringPrefixStartEnd(String self, String substr, int start, int end) {
577577
int len = self.length();
578-
return doIt(self, substr, adjustStart(start, len), adjustStart(end, len));
578+
return doIt(self, substr, adjustStart(start, len), adjustEnd(end, len));
579579
}
580580

581581
@Specialization
@@ -592,7 +592,7 @@ boolean doStringPrefix(String self, String substr, @SuppressWarnings("unused") P
592592
@Specialization
593593
boolean doTuplePrefixStartEnd(String self, PTuple substrs, int start, int end) {
594594
int len = self.length();
595-
return doIt(self, substrs, adjustStart(start, len), adjustStart(end, len));
595+
return doIt(self, substrs, adjustStart(start, len), adjustEnd(end, len));
596596
}
597597

598598
@Specialization

0 commit comments

Comments
 (0)