[ 
https://issues.apache.org/activemq/browse/CAMEL-298?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=42332#action_42332
 ] 

Hadrian Zbarcea commented on CAMEL-298:
---------------------------------------

Claus, I worked on this issue a while ago and the patch afair was incomplete.  
By now it's quite out of date, so the work would probably need to be redone.  
That aside as this incurs api changes I am not sure when would be the right 
time for it.  Thoughts?

> Remove tangle between impl and impl.converter
> ---------------------------------------------
>
>                 Key: CAMEL-298
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-298
>             Project: Apache Camel
>          Issue Type: Improvement
>          Components: camel-core
>            Reporter: Christian Schneider
>            Assignee: Claus Ibsen
>             Fix For: 1.4.0
>
>         Attachments: after patch no tangle.png, before patch.png
>
>
> Currently impl and impl.converter reference each other. If possible such 
> cycles should be avoided.
> When I looked into it I found that impl.converter references only a single 
> class CachingInjector. No other class in camel-core seems to use this class. 
> So my suggestion is to move CachingInjector to impl.converter. So the cycle i 
> removed. As far as I can see this should cause no ther issues.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to