struts-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Christian Grobmeier <grobme...@gmail.com>
Subject Re: Struts 2.3.13 ready to test
Date Wed, 27 Mar 2013 19:54:13 GMT
Sorry Maurizio,

can you elaborate a little more on the -1?
I can't see how our suggested fix for the stacktrace does in any way
interfere with the issue you mentioned. Maybe I miss the point.

Cheers,
Christian



On Wed, Mar 27, 2013 at 5:29 PM, Maurizio Cucchiara
<mcucchiara@apache.org> wrote:
> -1
>
> see https://issues.apache.org/jira/browse/WW-4030
>
>
> Twitter     :http://www.twitter.com/m_cucchiara
> G+          :https://plus.google.com/107903711540963855921
> Linkedin    :http://www.linkedin.com/in/mauriziocucchiara
> VisualizeMe: http://vizualize.me/maurizio.cucchiara?r=maurizio.cucchiara
>
> Maurizio Cucchiara
>
>
> On 27 March 2013 13:07, Christian Grobmeier <grobmeier@gmail.com> wrote:
>> On Wed, Mar 27, 2013 at 12:56 PM, Lukasz Lenart <lukaszlenart@apache.org> wrote:
>>> 2013/3/27 Christian Grobmeier <grobmeier@gmail.com>:
>>>> Its also there with 2.3.12, but not with 2.3.8.
>>>>
>>>> I tried to track it down and I think its because of rv1433644. Around
>>>> line it previously showed:
>>>>
>>>> LOG.debug("Unable to load optional class: " + ex);
>>>>
>>>> Now it is:
>>>>
>>>> LOG.debug("Unable to load optional class: #0", ex, impl);
>>>>
>>>> Not sure if we need the full exception stacktrace when we didn't find
>>>> an optional class.
>>>> if you agree, I change it too:
>>>>
>>>> LOG.debug("Unable to load optional class: #0", impl);
>>>>
>>>> Wdyt?
>>>
>>> You are right, nice work :-)
>>
>> Committed with rv1461544 (WW-4029), thanks for the help :-)
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
>> For additional commands, e-mail: dev-help@struts.apache.org
>>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
> For additional commands, e-mail: dev-help@struts.apache.org
>



-- 
http://www.grobmeier.de
https://www.timeandbill.de

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


Mime
View raw message