I vaguely remember a while back an attempt to make packages, etc. reloadable.
This was before we created an OSGi plugin, so it might not be relevant any
more.
IMO, between the OSGi plugin and the new spring enhancements you added, there
is probably no need to worry about being able to reload the container. In
fact, if we have tweaks like this, meant to support the ability to reload
configuration, that slow the framework down, we should probably get rid of
them.
-Wes
On Tuesday 21 July 2009 08:56:57 pm Musachy Barroso wrote:
> This code in DefaultActionInvocation.createResult:
>
> ActionConfig config = proxy.getConfig();
> Map<String, ResultConfig> results = config.getResults();
>
> ResultConfig resultConfig = null;
>
> synchronized (config) {
> try {
> resultConfig = results.get(resultCode);
> } catch (NullPointerException e) {
> // swallow
> }
> if (resultConfig == null) {
> // If no result is found for the given resultCode, try
> to get a wildcard '*' match.
> resultConfig = results.get("*");
> }
> }
>
> I can't see why wee need to synchronize that block...any ideas?
>
> musachy
--
Wes Wannemacher
Author - Struts 2 In Practice
Includes coverage of Struts 2.1, Spring, JPA, JQuery, Sitemesh and more
http://www.manning.com/wannemacher
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
