2018-02-20 10:21 GMT+01:00 Yasser Zamani <yasserzam...@apache.org>:
>> This is not needed as the reload check was moved into
>> FileManager#fileNeedsReloading methods - see usage of those methods.
>
> Yes I saw. However, FileManager#fileNeedsReloading method depends on if
> `revision = files.get` is null or not i.e. depends on if the file is
> under monitor or not, and as loadFile always calls monitorFile then
> currently, every loaded file is under monitor regardless of if user has
> set `struts.configuration.xml.reload` constant to true or not.

but FileManager#fileNeedsReloading won't be called if
"struts.configuration.xml.reload" is set to false, see
ConfigurationManager#conditionalReload


Regards
-- 
Ɓukasz
+ 48 606 323 122 http://www.lenart.org.pl/

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org
For additional commands, e-mail: dev-h...@struts.apache.org

Reply via email to