> > question is only if this is really desired...
> >
> > Berin, how did you wanted this to work?
> > Do you want me to provide a patch?
>
>
> Go for it.
...so I assume this was desired :-/
Berin, are you really sure about this?
I attached *both* patches. For the Resource as well as for ActiveMonitor.
But I am in favor for patching the Resource *only*...
Would be great if someone could apply one of them...
Thanks
--
Torsten
Index: Resource.java
===================================================================
RCS file:
/home/cvspublic/jakarta-avalon-excalibur/all/src/java/org/apache/avalon/excalibur/monitor/Resource.java,v
retrieving revision 1.4
diff -u -r1.4 Resource.java
--- Resource.java 26 Mar 2002 10:45:22 -0000 1.4
+++ Resource.java 22 Apr 2002 14:50:31 -0000
@@ -62,7 +62,7 @@
{
long lastModified = this.lastModified();
- if( lastModified > time )
+ if( lastModified > m_previousModified )
{
m_eventSupport.firePropertyChange( Resource.MODIFIED,
new Long( m_previousModified ),
Index: ActiveMonitor.java
===================================================================
RCS file:
/home/cvspublic/jakarta-avalon-excalibur/all/src/java/org/apache/avalon/excalibur/monitor/ActiveMonitor.java,v
retrieving revision 1.9
diff -u -r1.9 ActiveMonitor.java
--- ActiveMonitor.java 24 Mar 2002 09:56:39 -0000 1.9
+++ ActiveMonitor.java 22 Apr 2002 14:37:05 -0000
@@ -101,6 +101,10 @@
m_monitorThread.setDaemon( true );
m_monitorThread.setPriority( Thread.MIN_PRIORITY );
m_monitorThread.start();
+ if( getLogger().isDebugEnabled() )
+ {
+ getLogger().debug("start monitoring");
+ }
}
public final void stop()
@@ -108,6 +112,10 @@
{
m_keepRunning = false;
m_monitorThread.join();
+ if( getLogger().isDebugEnabled() )
+ {
+ getLogger().debug("stop monitoring");
+ }
}
/**
@@ -191,7 +199,11 @@
for( int i = 0; i < resources.length; i++ )
{
- resources[ i ].testModifiedAfter( currentTestTime );
+ if( getLogger().isDebugEnabled() )
+ {
+ getLogger().debug("checking " + resources[ i ].getResourceKey() );
+ }
+ resources[ i ].testModifiedAfter( resources[i].m_previousModified );
}
}
}
--
To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
