Re: svn commit: r1349901 - in /camel/trunk/components/camel-cxf/src: main/java/org/apache/camel/component/cxf/converter/ test/java/org/apache/camel/component/cxf/ test/resources/org/apache/camel/compo

2012-06-14 Thread Willem Jiang
Hi Claus, Thanks for the review, I just update the code by removing the try ... catch codes. On Thu Jun 14 11:34:45 2012, Claus Ibsen wrote: Hi In the converter code, you do a try .. catch and then swallow the caused exception. You should not do this, but throw the exception. And remove the

Re: svn commit: r1349901 - in /camel/trunk/components/camel-cxf/src: main/java/org/apache/camel/component/cxf/converter/ test/java/org/apache/camel/component/cxf/ test/resources/org/apache/camel/compo

2012-06-13 Thread Claus Ibsen
Hi In the converter code, you do a try .. catch and then swallow the caused exception. You should not do this, but throw the exception. And remove the logging. On Wed, Jun 13, 2012 at 5:20 PM, wrote: > Author: ningjiang > Date: Wed Jun 13 15:20:35 2012 > New Revision: 1349901 > > URL: http://s