harmony-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Tim Ellison (JIRA)" <j...@apache.org>
Subject [jira] Assigned: (HARMONY-547) [classlib][jndi] dead code in javax.naming.spi.DirectoryManager
Date Wed, 03 Jan 2007 13:43:27 GMT

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

Tim Ellison reassigned HARMONY-547:
-----------------------------------

    Assignee: Tim Ellison

> [classlib][jndi] dead code in javax.naming.spi.DirectoryManager
> ---------------------------------------------------------------
>
>                 Key: HARMONY-547
>                 URL: https://issues.apache.org/jira/browse/HARMONY-547
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib
>            Reporter: Mikhail Loenko
>         Assigned To: Tim Ellison
>         Attachments: Harmony-547.patch
>
>
> There are two dead blocks in DirectoryManager:
>         Context nextContext = null;
>         try {
>             nextContext = NamingManager.getContinuationContext(cpe);
>         } catch (CannotProceedException e) {
>             // tolerate CannotProceedException here
>         }
>         
>         // if it is a DirContext
>         if (nextContext instanceof DirContext) {
>             // return as DirContext
>             return (DirContext) nextContext;
>         } else if (nextContext instanceof Context) { <<<<<<<<<<
always true
>             // in case it's Context but not DirContext, wrap it as DirContext and return
>             return new Context2DirContextWrapper(nextContext, cpe);
>         } else {  <<<<<<<<<<<<<< dead block
>             // return a dummy DirContext, even if no context is obtained
>             return new Context2DirContextWrapper(null, cpe);
>         }
> and
>             if (ctx instanceof Context) {   <<<<<<<<< always
true
>                 return ctx;
>             }
>             cpe.fillInStackTrace();     <<<<<<<<<<<<<<<
dead block
>             throw cpe;
> I'm not sure how to fix it. Seems like special care for null should be done

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

        

Mime
View raw message