harmony-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alexei Zakharov (JIRA)" <j...@apache.org>
Subject [jira] Resolved: (HARMONY-2512) [classlib][swing] javax.swing.plaf.basic.BasicPopupMenuUI.isPopupTrigger(null) does not throw unspecified NPE
Date Mon, 12 Feb 2007 12:05:06 GMT

     [ https://issues.apache.org/jira/browse/HARMONY-2512?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Alexei Zakharov resolved HARMONY-2512.
--------------------------------------

    Resolution: Fixed

Thanks Denis and Sergey. The patch was applied at the revision 506428. Please verify that
it was applied as expected.

> [classlib][swing] javax.swing.plaf.basic.BasicPopupMenuUI.isPopupTrigger(null) does not
throw unspecified NPE
> -------------------------------------------------------------------------------------------------------------
>
>                 Key: HARMONY-2512
>                 URL: https://issues.apache.org/jira/browse/HARMONY-2512
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib
>            Reporter: Denis Kishenko
>         Assigned To: Alexei Zakharov
>         Attachments: Harmony-2512-BasicPopupMenuUI.patch, Harmony-2512-BasicPopupMenuUITest.patch
>
>
> There is no mention of any exceptions in the specification. Harmony does not throw unspecified
NPE for javax.swing.plaf.basic.BasicPopupMenuUI.isPopupTrigger(null) while RI does. 
> Compatibility issue.
> Test for reproducing:
> import junit.framework.TestCase;
> import javax.swing.*;
> import javax.swing.plaf.basic.BasicPopupMenuUI;
> public class Test extends TestCase {       
>     public void testcase1() {
>         BasicPopupMenuUI m = new BasicPopupMenuUI();
>         try {
>             m.isPopupTrigger(null);             
>             fail("NPE should be thrown");
>         } catch ( NullPointerException e) { 
>         }
>     }
> }
> Output on Sun 1.5:
> ==================
> .
> Time: 0.282
> OK (1 test)
> Output on Harmony:
> ==================
> .F
> Time: 0.015
> There was 1 failure:
> 1) testcase1(Test)junit.framework.AssertionFailedError: NPE should be thrown
>         at Test.testcase1(Test.java:27)
>         at java.lang.reflect.VMReflection.invokeMethod(Native Method)
> FAILURES!!!
> Tests run: 1,  Failures: 1,  Errors: 0

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message