Exchange route refactor patch - comments?

[Kim van der Riet <ki...@redhat.com>]

This patch is a small Exchange refactor around the route() method. It
was part of r813825 which was subsequently rolled back out. The current
solution to QPID-2102 does not require this part of that original patch,
but as this is hopefully a useful cleanup of some code duplication, I
submit it here for comment before checking it in.

Comments welcome.

RE: Exchange route refactor patch - comments?

[Steve Huston <sh...@riverace.com>]

Hi Kim,

> This patch is a small Exchange refactor around the route() method.
It
> was part of r813825 which was subsequently rolled back out. 
> The current
> solution to QPID-2102 does not require this part of that 
> original patch,
> but as this is hopefully a useful cleanup of some code duplication,
I
> submit it here for comment before checking it in.
> 
> Comments welcome.

I'm not an expert in this area of the code, but it looks like a nice
refactoring of a lot of nearly duplicate code sections. Looks good to
me.

-Steve


---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:dev-subscribe@qpid.apache.org