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] Updated: (HARMONY-547) [classlib][jndi] dead code in javax.naming.spi.DirectoryManager
Date Thu, 21 Dec 2006 09:40:23 GMT
     [ http://issues.apache.org/jira/browse/HARMONY-547?page=all ]

Alexei Zakharov updated HARMONY-547:
------------------------------------

    Summary: [classlib][jndi] dead code in javax.naming.spi.DirectoryManager  (was: dead code
in javax.naming.spi.DirectoryManager)

> [classlib][jndi] dead code in javax.naming.spi.DirectoryManager
> ---------------------------------------------------------------
>
>                 Key: HARMONY-547
>                 URL: http://issues.apache.org/jira/browse/HARMONY-547
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib
>            Reporter: Mikhail Loenko
>
> 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: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Mime
View raw message