excalibur-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Berin Loritsch <blorit...@d-haven.org>
Subject Re: D-Haven-event bug in current Fortress trunk build.
Date Thu, 29 Jul 2004 15:28:58 GMT
Leif Mortenson wrote:

> I did not see where D-Haven project bugs should be posted. It sounds
> like the
> members are the same, so this should get into the right hands :-)
> 
> If Fortress has any problems initializing any of its components then it
> is designed
> to call CommandFailureHandler failure. This defaults to the
> FortressCommandFailureHandler but it is possible to override it with one
> of your
> own. (As I have done).
> 
> The current trunk build now makes use of the d-haven-event jar and this
> has been
> broken.
> 
> Digging into the d-haven source, I see the problem:
> 
> When the DefaultCommandManager is instantiated, it initially sets its
> internal
> CommandFailureHandler to an instance of DefaultCommandFailureHandler.
> Then within the constructor, it immediately uses that handler to instantiate
> a new CommandEventHandler.
> 
> The DefaultCommandManager class has a setCommandFailureHandler method
> which is being called by Fortress after the constructor completes. But the
> problem is that it is already too late to be used. So my custom handler, and
> even the default FortressCommandFailureHandler is never used.
> 
> Either the instantiation of the CommandEventHandler is going to have to be
> moved, or a way of changing its CommandFailureHander is going to have to
> be introduced.

This bug is also now fixed.

The next bug fix release should be posted sometime after lunch (starting
now).  The build went successfully, and the tests make sure things are
happening well.

-- 

"Programming today is a race between software engineers striving to
build bigger and better idiot-proof programs, and the Universe trying to
produce bigger and better idiots. So far, the Universe is winning."
                - Rich Cook

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@excalibur.apache.org
For additional commands, e-mail: dev-help@excalibur.apache.org
Apache Excalibur Project -- URL: http://excalibur.apache.org/


Mime
View raw message