geronimo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
Subject jira Closed: (GERONIMO-144) [PATCH] Invalid condition in o.a.g.gbean.jmx.CollectionProxy.removeTarget
Date Thu, 15 Jan 2004 00:51:52 GMT

   The following issue has been closed.

View the issue:

Here is an overview of the issue:
        Key: GERONIMO-144
    Summary: [PATCH] Invalid condition in o.a.g.gbean.jmx.CollectionProxy.removeTarget
       Type: Bug

     Status: Closed
   Priority: Major
 Resolution: FIXED

    Project: Apache Geronimo

   Assignee: Dain Sundstrom
   Reporter: Ed Letifov

    Created: Wed, 14 Jan 2004 2:12 PM
    Updated: Wed, 14 Jan 2004 4:51 PM

I think there is a bug in the code (the condition should be reversed and I would suggest dropping
containsKey+remove it in favor of remove+check results for being not null):
    public synchronized void removeTarget(ObjectName target) {
        // if this is one of our existing targets target...
        if (!proxies.containsKey(target)) {
            ProxyMethodInterceptor interceptor = (ProxyMethodInterceptor) interceptors.remove(target);
            if (interceptor != null) {

This message is automatically generated by JIRA.

If you think it was sent incorrectly contact one of the administrators:

If you want more information on JIRA, or have a bug to report see:

View raw message