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:[email protected]
