OpenJDK throws StringIndexOutOfBoundsException and so should we.

Signed-off-by: Pekka Enberg <penb...@kernel.org>
---
 ChangeLog             |    6 ++++++
 java/lang/String.java |    4 ++++
 2 files changed, 10 insertions(+), 0 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index cce9803..0a867cc 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,11 @@
 2012-03-15  Pekka Enberg  <penb...@kernel.org>
 
+       * java/lang/String.java:
+       (codePointAt): Fix exception type.
+       (codePointBefore): Fix exception type.
+
+2012-03-15  Pekka Enberg  <penb...@kernel.org>
+
        * java/util/Formatter.java:
        (format): Fix NPE errors.
 
diff --git a/java/lang/String.java b/java/lang/String.java
index 45c0daf..eb713ce 100644
--- a/java/lang/String.java
+++ b/java/lang/String.java
@@ -705,6 +705,8 @@ public final class String
    */
   public synchronized int codePointAt(int index)
   {
+    if (index < 0 || index >= count)
+      throw new StringIndexOutOfBoundsException(index);
     // Use the CharSequence overload as we get better range checking
     // this way.
     return Character.codePointAt(this, index);
@@ -722,6 +724,8 @@ public final class String
    */
   public synchronized int codePointBefore(int index)
   {
+    if (index < 0 || index >= count)
+      throw new StringIndexOutOfBoundsException(index);
     // Use the CharSequence overload as we get better range checking
     // this way.
     return Character.codePointBefore(this, index);
-- 
1.7.6.5


Reply via email to