You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@avalon.apache.org by bu...@apache.org on 2002/12/16 10:37:35 UTC
DO NOT REPLY [Bug 15387] New: -
handleEvents( Object[] elements ) in CommandManager.java
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=15387>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND
INSERTED IN THE BUG DATABASE.
http://nagoya.apache.org/bugzilla/show_bug.cgi?id=15387
handleEvents( Object[] elements ) in CommandManager.java
Summary: handleEvents( Object[] elements ) in CommandManager.java
Product: Avalon
Version: unspecified
Platform: All
OS/Version: All
Status: NEW
Severity: Major
Priority: Other
Component: Excalibur
AssignedTo: avalon-dev@jakarta.apache.org
ReportedBy: geb@263.net
in line 258, code is like below:
for( int i = 0; i < size; i++ )
{
DelayedCommandInfo command = (DelayedCommandInfo)m_delayedCommands.remove();
if( System.currentTimeMillis() >= command.m_nextRunTime )
{
// do delay command process
....
}
}
but if System.currentTimeMillis() is less than command.m_nextRunTime, command
is not process and do not put i back to m_delayedCommands. it will cause repeat
command lost. i think it should add m_delayedCommands.add(command) after if
block.
--
To unsubscribe, e-mail: <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>