DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=34289>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=34289


[EMAIL PROTECTED] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #15105|0                           |1
        is obsolete|                            |




------- Additional Comments From [EMAIL PROTECTED]  2005-05-31 12:29 -------
Created an attachment (id=15228)
 --> (http://issues.apache.org/bugzilla/attachment.cgi?id=15228&action=view)
Updated patch for FileChangedReloadingStrategy plus unit test

Patch updated to incorporate changes suggested by Jorge Ferrer.

FileChangedReloadingStrategy will now correctly update its lastChanged field,
so that hasChanged() need not be called every time reloadingRequired() is
invoked.

I also included the proposed functionality that if a configuration file is
loaded from a jar, the jar file itself should be the target for the strategy.
However IMO the containerFileURL() method seems to be very specific to the
reloading strategy, so I placed the code directly in
FileChangedReloadingStrategy. If we later find out that this stuff is generally
useful, we can still move it to ConfigurationUtils and make it public.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

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

Reply via email to