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/03/17 10:34:30 UTC

svn commit: r519272 - /harmony/enhanced/classlib/trunk/modules/swing/src/main/java/common/javax/swing/plaf/basic/BasicFileChooserUI.java

Author: apetrenko
Date: Sat Mar 17 02:34:29 2007
New Revision: 519272

URL: http://svn.apache.org/viewvc?view=rev&rev=519272
Log:
This patch is a correct fix for HARMONY-2606 "[classlib][swing][plaf] javax.swing.plaf.basic.BasicFileChooserUI.ensureFileIsVisible(..) throws unexpected NPE" and it also removes a regression introduced by previous patch

Modified:
    harmony/enhanced/classlib/trunk/modules/swing/src/main/java/common/javax/swing/plaf/basic/BasicFileChooserUI.java

Modified: harmony/enhanced/classlib/trunk/modules/swing/src/main/java/common/javax/swing/plaf/basic/BasicFileChooserUI.java
URL: http://svn.apache.org/viewvc/harmony/enhanced/classlib/trunk/modules/swing/src/main/java/common/javax/swing/plaf/basic/BasicFileChooserUI.java?view=diff&rev=519272&r1=519271&r2=519272
==============================================================================
--- harmony/enhanced/classlib/trunk/modules/swing/src/main/java/common/javax/swing/plaf/basic/BasicFileChooserUI.java (original)
+++ harmony/enhanced/classlib/trunk/modules/swing/src/main/java/common/javax/swing/plaf/basic/BasicFileChooserUI.java Sat Mar 17 02:34:29 2007
@@ -617,8 +617,9 @@
     }
 
     public void ensureFileIsVisible(final JFileChooser fc, final File f) {
-        fileChooser = fc;  
-        fileChooser.ensureFileIsVisible(f);
+        if (fileChooser != null) {  
+            fileChooser.ensureFileIsVisible(f);
+        }
     }
 
     public JFileChooser getFileChooser() {