You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by ap...@apache.org on 2007/01/13 20:10:06 UTC

svn commit: r495947 - in /harmony/enhanced/classlib/trunk/modules/swing/src: main/java/common/javax/swing/text/DefaultCaret.java test/api/java/common/javax/swing/text/DefaultCaretTest.java

Author: apetrenko
Date: Sat Jan 13 11:10:05 2007
New Revision: 495947

URL: http://svn.apache.org/viewvc?view=rev&rev=495947
Log:
Patch for HARMONY-1750 "[classlib][swing]javax.swing.text.DefaultCaret.install(null) does not throw unspecified NPE"

Modified:
    harmony/enhanced/classlib/trunk/modules/swing/src/main/java/common/javax/swing/text/DefaultCaret.java
    harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/text/DefaultCaretTest.java

Modified: harmony/enhanced/classlib/trunk/modules/swing/src/main/java/common/javax/swing/text/DefaultCaret.java
URL: http://svn.apache.org/viewvc/harmony/enhanced/classlib/trunk/modules/swing/src/main/java/common/javax/swing/text/DefaultCaret.java?view=diff&rev=495947&r1=495946&r2=495947
==============================================================================
--- harmony/enhanced/classlib/trunk/modules/swing/src/main/java/common/javax/swing/text/DefaultCaret.java (original)
+++ harmony/enhanced/classlib/trunk/modules/swing/src/main/java/common/javax/swing/text/DefaultCaret.java Sat Jan 13 11:10:05 2007
@@ -674,6 +674,9 @@
      * selectionColor.
      */
     public void install(final JTextComponent c) {
+        if (c == null) { // Fix for HARMONY-1750, for compatiblity with RI
+            throw new NullPointerException("component is null");
+        }
         if (!setComponent(c)) {
             return;
         }

Modified: harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/text/DefaultCaretTest.java
URL: http://svn.apache.org/viewvc/harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/text/DefaultCaretTest.java?view=diff&rev=495947&r1=495946&r2=495947
==============================================================================
--- harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/text/DefaultCaretTest.java (original)
+++ harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/text/DefaultCaretTest.java Sat Jan 13 11:10:05 2007
@@ -226,6 +226,13 @@
                 + jta.getListeners(FocusListener.class).length
                 + jta.getPropertyChangeListeners().length;
         assertEquals(tCompListenersCount, tCompListenersCountCurrent - 4);
+
+        try { // Regression test for HARMONY-1750
+            new DefaultCaret().install(null);
+            fail("NullPointerException should have been thrown");
+        } catch (NullPointerException e) {
+            // Expected
+        }
     }
 
     /*