[jira] Commented: (CAMEL-298) Remove tangle between impl and impl.converter

["Christian Schneider (JIRA)" <ji...@apache.org>]

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

Christian Schneider commented on CAMEL-298:
-------------------------------------------

Should I append a patch for this?

> 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
>             Fix For: 1.3.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.