----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8675/#review14692 -----------------------------------------------------------
Goody, goody, less code to maintain! I suggest removing stuff in fairly small (but consistent) chunks in case we need to revert one of the removals for whatever reason. /trunk/qpid/cpp/src/qpid/broker/Connection.cpp <https://reviews.apache.org/r/8675/#comment31152> You've removed the comment but not changed the behaviour, that doesn't seem right. /trunk/qpid/cpp/src/qpid/broker/Connection.cpp <https://reviews.apache.org/r/8675/#comment31153> Again removed comment but not changed behaviour. /trunk/qpid/cpp/src/qpid/broker/Connection.cpp <https://reviews.apache.org/r/8675/#comment31154> And again. - Andrew Stitcher On Dec. 18, 2012, 10:11 p.m., Alan Conway wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/8675/ > ----------------------------------------------------------- > > (Updated Dec. 18, 2012, 10:11 p.m.) > > > Review request for qpid and Andrew Stitcher. > > > Description > ------- > > This is an initial pass, there is more cluster-related code that can be > removed. > > > Diffs > ----- > > /trunk/qpid/cpp/src/qpid/acl/AclConnectionCounter.cpp 1423689 > /trunk/qpid/cpp/src/qpid/agent/ManagementAgentImpl.h 1423689 > /trunk/qpid/cpp/src/qpid/amqp_0_10/Connection.h 1423689 > /trunk/qpid/cpp/src/qpid/amqp_0_10/Connection.cpp 1423689 > /trunk/qpid/cpp/src/qpid/broker/Broker.h 1423689 > /trunk/qpid/cpp/src/qpid/broker/Broker.cpp 1423689 > /trunk/qpid/cpp/src/qpid/broker/Connection.h 1423689 > /trunk/qpid/cpp/src/qpid/broker/Connection.cpp 1423689 > /trunk/qpid/cpp/src/qpid/broker/ConnectionState.h 1423689 > /trunk/qpid/cpp/src/qpid/broker/DtxManager.h 1423689 > /trunk/qpid/cpp/src/qpid/broker/DtxWorkRecord.h 1423689 > /trunk/qpid/cpp/src/qpid/broker/DtxWorkRecord.cpp 1423689 > /trunk/qpid/cpp/src/qpid/broker/Exchange.cpp 1423689 > /trunk/qpid/cpp/src/qpid/broker/Link.h 1423689 > /trunk/qpid/cpp/src/qpid/broker/Link.cpp 1423689 > /trunk/qpid/cpp/src/qpid/broker/LinkRegistry.h 1423689 > /trunk/qpid/cpp/src/qpid/broker/LinkRegistry.cpp 1423689 > /trunk/qpid/cpp/src/qpid/broker/MessageGroupManager.h 1423689 > /trunk/qpid/cpp/src/qpid/broker/MessageGroupManager.cpp 1423689 > /trunk/qpid/cpp/src/qpid/broker/Messages.h 1423689 > /trunk/qpid/cpp/src/qpid/broker/Queue.cpp 1423689 > /trunk/qpid/cpp/src/qpid/broker/QueueFlowLimit.h 1423689 > /trunk/qpid/cpp/src/qpid/broker/QueueFlowLimit.cpp 1423689 > /trunk/qpid/cpp/src/qpid/broker/SessionHandler.h 1423689 > /trunk/qpid/cpp/src/qpid/broker/SessionHandler.cpp 1423689 > /trunk/qpid/cpp/src/qpid/broker/SessionState.h 1423689 > /trunk/qpid/cpp/src/qpid/broker/SessionState.cpp 1423689 > /trunk/qpid/cpp/src/qpid/broker/StatefulQueueObserver.h 1423689 > /trunk/qpid/cpp/src/qpid/management/ManagementAgent.h 1423689 > /trunk/qpid/cpp/src/qpid/management/ManagementAgent.cpp 1423689 > /trunk/qpid/cpp/src/qpid/sys/Timer.h 1423689 > /trunk/qpid/cpp/src/qpid/sys/Timer.cpp 1423689 > /trunk/qpid/cpp/xml/cluster.xml 1423689 > /trunk/qpid/doc/book/src/cpp-broker/AMQP-Messaging-Broker-CPP-Book.xml > 1423689 > /trunk/qpid/doc/book/src/cpp-broker/Active-Active-Cluster.xml 1423689 > /trunk/qpid/doc/book/src/cpp-broker/Active-Passive-Cluster.xml 1423689 > > Diff: https://reviews.apache.org/r/8675/diff/ > > > Testing > ------- > > > Thanks, > > Alan Conway > >