diff options
author | Gordon Sim <gsim@apache.org> | 2007-07-17 08:28:48 +0000 |
---|---|---|
committer | Gordon Sim <gsim@apache.org> | 2007-07-17 08:28:48 +0000 |
commit | 87663b00fb0aa54d3df740c14c8c925c0aaee227 (patch) | |
tree | cd37e01bcc9839fb864526148af77cd836612f11 /qpid/cpp/src/qpid/broker/HandlerImpl.h | |
parent | ccabc769cd8f7073bb32854c209aee6d4a8fea9d (diff) | |
download | qpid-python-87663b00fb0aa54d3df740c14c8c925c0aaee227.tar.gz |
Some refactoring towards a more decoupled handler chain structure:
* Connection no longer depends on Channel; it contains a map of
FrameHandler::Chains. (The construction of the chains still refers
to specific handlers).
* Channel is no longer tied to ChannelAdapter through inheritance. The
former is independent of any particular handler chain or protocol
version, the latter is still used by ConnectionAdapter and
SemanticHandler in the 0-9 chain.
* A DeliveryAdapter interface has been introduced as part of the
separation of ChannelAdapter from Channel. This is intended to adapt
from a version independent core to version specific mechanisms for
sending messages. i.e. it fulfills the same role for outputs that
e.g. BrokerAdapter does for inputs. (Its not perfect yet by any
means but is a step on the way to the correct model I think).
* The connection related methods sent over channel zero are
implemented in their own adapter (ConnectionAdapter), and are
entirely separate from the semantic layer. The channel control
methods are still bundled with the proper semantic layer methods;
they too can be separated but would have to share the request id
with the semantic method handler due to the nature of the 0-9 WIP.
git-svn-id: https://svn.apache.org/repos/asf/incubator/qpid/trunk@556846 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'qpid/cpp/src/qpid/broker/HandlerImpl.h')
-rw-r--r-- | qpid/cpp/src/qpid/broker/HandlerImpl.h | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/qpid/cpp/src/qpid/broker/HandlerImpl.h b/qpid/cpp/src/qpid/broker/HandlerImpl.h index 008be10867..96bf065062 100644 --- a/qpid/cpp/src/qpid/broker/HandlerImpl.h +++ b/qpid/cpp/src/qpid/broker/HandlerImpl.h @@ -40,12 +40,13 @@ class Connection; */ struct CoreRefs { - CoreRefs(Channel& ch, Connection& c, Broker& b) - : channel(ch), connection(c), broker(b), proxy(ch) {} + CoreRefs(Channel& ch, Connection& c, Broker& b, framing::ChannelAdapter& a) + : channel(ch), connection(c), broker(b), adapter(a), proxy(a) {} Channel& channel; Connection& connection; Broker& broker; + framing::ChannelAdapter& adapter; framing::AMQP_ClientProxy proxy; /** |