It is a good improvement, +1 from me too. Christian, go for it! Hadrian On Nov 3, 2010, at 11:40 PM, Willem Jiang wrote:
> It makes sense, +1 for the change. > > On 11/4/10 7:28 AM, Christian Müller wrote: >> Hello Rafal! >> >> Please provide more details like the Camel version you are using in >> further questions. Please have a look here: >> http://camel.apache.org/how-can-i-get-help.html >> >> However, I assume you have a String as payload in your message. The >> default Camel type converter mechanism is able to convert this payload >> also into an InputStream. This is the reason, why this is ambiguous for >> Camel... >> >> I wondering, if we SHOULD make Camel a bit smarter here. I made a patch >> which fixes this issue. We check whether the message payload is an >> instance of the bean method argument. If so, we will use this method and >> don't convert the body. Any objections? >> >> Cheers, >> Christian > > > -- > Willem > ---------------------------------- > FuseSource > Web: http://www.fusesource.com > Blog: http://willemjiang.blogspot.com (English) > http://jnn.javaeye.com (Chinese) > Twitter: willemjiang
