Mailing-List: contact camel-dev-help@activemq.apache.org; run by ezmlm
Precedence: bulk
Reply-To: camel-dev@activemq.apache.org
Message-ID: <2251575.1200759981813.JavaMail.jira@brutus>
Date: Sat, 19 Jan 2008 08:26:21 -0800 (PST)
From: "Christian Schneider (JIRA)" <jira@apache.org>
To: camel-dev@activemq.apache.org
Subject: [jira] Commented: (CAMEL-298) Remove tangle between impl and
 impl.converter
In-Reply-To: <27421072.1200759862086.JavaMail.jira@brutus>
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit


    [ 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.