You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by "Alexey A. Ivanov (JIRA)" <ji...@apache.org> on 2006/11/15 15:02:38 UTC

[jira] Commented: (HARMONY-2198) [classlib][swing] compatibility: j.s.text.GapContent.replace is not compatible with RI

    [ http://issues.apache.org/jira/browse/HARMONY-2198?page=comments#action_12450032 ] 
            
Alexey A. Ivanov commented on HARMONY-2198:
-------------------------------------------

The new tests reveal another incompatibility in GapContent.insertString: see testInsertStringInvalidNull().
In case 

gv.insertString(-1, null);

Harmony throws NPE whereas RI throw BadLocationException. The difference is merely in the order of checks/usages, and it's hard to make Harmony compatible without performing redundant checks. (See the similar test testInsertStringNull.)

> [classlib][swing] compatibility: j.s.text.GapContent.replace is not compatible with RI
> --------------------------------------------------------------------------------------
>
>                 Key: HARMONY-2198
>                 URL: http://issues.apache.org/jira/browse/HARMONY-2198
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib, Non-bug differences from RI, build - test - ci
>            Reporter: Alexey A. Ivanov
>         Attachments: H2189-GapContent_GapVectorTest.patch, H2198-GapContent.patch
>
>
> Harmony implementation of javax.swing.text.GapContent.replace() is not compatible with RI.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira