Leo Sutic wrote:

>Realized that I made the method public by mistake.
>
>    /**
>     * Method to assume ownership of one of the managers the
>     * <code>ContextManager</code> created.  Ownership means that the
>     * <code>ContextManager</code> is responsible for destroying the
>     * manager when the <code>ContextManager</code> is destroyed.
>     *
>  
>
>>>> * FIXME: Should this really be public?  Also, we should throw a
>>>> *        NullPointerException instead.
>>>>        
>>>>
>     *
>     * @param object  The object being claimed
>     *
>     * @throws IllegalArgumentException if the object is null.
>     */
>    public void assumeOwnership( Object o )
>
>Ooops. Not supposed to be public - supposed to be private.
>
>Do we just change it (any code using it deserves to get broken)
>or leave it?
>
>+1 for changing access modifier to private. 
>  
>

+1

Fortress has not gone through a formal realease and the status is well 
documented on the home page:

  " This package is under development, and the API is not guaranteed
   to be the same when it is ready for release. You can find this in the
   excalibur-fortress-1.0.jar file if you want to play with it. Do not 
blame
   us if the next release of Excalibur breaks your code if you use this
   package."


Cheers, Steve.


>/LS
>
>
>--
>To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
>For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
>
>  
>

-- 

Stephen J. McConnell

OSM SARL
digital products for a global economy
mailto:[EMAIL PROTECTED]
http://www.osm.net




--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to