You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by ha...@apache.org on 2010/10/07 08:05:38 UTC

svn commit: r1005334 - /camel/trunk/components/camel-cxf/src/main/java/org/apache/camel/component/cxf/CxfConsumer.java

Author: hadrian
Date: Thu Oct  7 06:05:38 2010
New Revision: 1005334

URL: http://svn.apache.org/viewvc?rev=1005334&view=rev
Log:
CAMEL-3201. Patch applied with thanks to Savas

Modified:
    camel/trunk/components/camel-cxf/src/main/java/org/apache/camel/component/cxf/CxfConsumer.java

Modified: camel/trunk/components/camel-cxf/src/main/java/org/apache/camel/component/cxf/CxfConsumer.java
URL: http://svn.apache.org/viewvc/camel/trunk/components/camel-cxf/src/main/java/org/apache/camel/component/cxf/CxfConsumer.java?rev=1005334&r1=1005333&r2=1005334&view=diff
==============================================================================
--- camel/trunk/components/camel-cxf/src/main/java/org/apache/camel/component/cxf/CxfConsumer.java (original)
+++ camel/trunk/components/camel-cxf/src/main/java/org/apache/camel/component/cxf/CxfConsumer.java Thu Oct  7 06:05:38 2010
@@ -126,7 +126,7 @@ public class CxfConsumer extends Default
             private Continuation getContinuation(Exchange cxfExchange) {
                 ContinuationProvider provider = 
                     (ContinuationProvider)cxfExchange.getInMessage().get(ContinuationProvider.class.getName());
-                return provider.getContinuation();
+                return provider == null ? null : provider.getContinuation();
             }
             
             private Object syncInvoke(Exchange cxfExchange) {



Re: svn commit: r1005334 - /camel/trunk/components/camel-cxf/src/main/java/org/apache/camel/component/cxf/CxfConsumer.java

Posted by Claus Ibsen <cl...@gmail.com>.
:) usually you would check for != instead of doing == null



On Thu, Oct 7, 2010 at 8:05 AM,  <ha...@apache.org> wrote:
> Author: hadrian
> Date: Thu Oct  7 06:05:38 2010
> New Revision: 1005334
>
> URL: http://svn.apache.org/viewvc?rev=1005334&view=rev
> Log:
> CAMEL-3201. Patch applied with thanks to Savas
>
> Modified:
>    camel/trunk/components/camel-cxf/src/main/java/org/apache/camel/component/cxf/CxfConsumer.java
>
> Modified: camel/trunk/components/camel-cxf/src/main/java/org/apache/camel/component/cxf/CxfConsumer.java
> URL: http://svn.apache.org/viewvc/camel/trunk/components/camel-cxf/src/main/java/org/apache/camel/component/cxf/CxfConsumer.java?rev=1005334&r1=1005333&r2=1005334&view=diff
> ==============================================================================
> --- camel/trunk/components/camel-cxf/src/main/java/org/apache/camel/component/cxf/CxfConsumer.java (original)
> +++ camel/trunk/components/camel-cxf/src/main/java/org/apache/camel/component/cxf/CxfConsumer.java Thu Oct  7 06:05:38 2010
> @@ -126,7 +126,7 @@ public class CxfConsumer extends Default
>             private Continuation getContinuation(Exchange cxfExchange) {
>                 ContinuationProvider provider =
>                     (ContinuationProvider)cxfExchange.getInMessage().get(ContinuationProvider.class.getName());
> -                return provider.getContinuation();
> +                return provider == null ? null : provider.getContinuation();
>             }
>
>             private Object syncInvoke(Exchange cxfExchange) {
>
>
>



-- 
Claus Ibsen
Apache Camel Committer

Author of Camel in Action: http://www.manning.com/ibsen/
Open Source Integration: http://fusesource.com
Blog: http://davsclaus.blogspot.com/
Twitter: http://twitter.com/davsclaus