diff options
| author | Alan Conway <aconway@apache.org> | 2008-10-25 01:55:06 +0000 |
|---|---|---|
| committer | Alan Conway <aconway@apache.org> | 2008-10-25 01:55:06 +0000 |
| commit | 57bd5193208b228c1088586917d7f43f13e0dd9a (patch) | |
| tree | 564d1aa0d13da985bd2159bbdd8d4b92be4016fb /cpp/src/qpid/client/HandlePrivate.h | |
| parent | d1239516d2cd33ceb90be7a74bd5ea73825c577e (diff) | |
| download | qpid-python-57bd5193208b228c1088586917d7f43f13e0dd9a.tar.gz | |
Client API change: Centralize access to subscription status, better control of acquire/accept.
client/AckPolicy: removed, functionality moved to Subscription and SubscriptionSettings
client/SubscriptionSettings: struct aggregates flow control & accept-acquire parameters for subscribe.
client/Subscription: represents active subscription. Query settings, unacked messages, manual accept/acquire
client/SubscriptionManager: use AcceptMode, AcquireMode enums rather than confusing bools.
Issues addressed by the change:
- old use of bool for acceptMode was inverted wrt AMQP enum values, bools are confusing.
- old AckPolicy was broken - not possible to access the instance associated with an active subscription
- old AckPolicy did not provide a way to do manual acquire, only accept.
- setting values on SubscriptionManager to apply to subsequent subscriptions is awkward & error-prone, now can use SubscriptionSettings to control on each subscribe individually.
- a subscription is a central concept in AMQP, it deserves to be a class. Subscription and SubscriptionSettings provides a single point for future expansion of interactions with a a Subscription.
git-svn-id: https://svn.apache.org/repos/asf/incubator/qpid/trunk/qpid@707808 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'cpp/src/qpid/client/HandlePrivate.h')
| -rw-r--r-- | cpp/src/qpid/client/HandlePrivate.h | 61 |
1 files changed, 61 insertions, 0 deletions
diff --git a/cpp/src/qpid/client/HandlePrivate.h b/cpp/src/qpid/client/HandlePrivate.h new file mode 100644 index 0000000000..488ce48075 --- /dev/null +++ b/cpp/src/qpid/client/HandlePrivate.h @@ -0,0 +1,61 @@ +#ifndef QPID_CLIENT_HANDLEPRIVATE_H +#define QPID_CLIENT_HANDLEPRIVATE_H + +/* + * + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agreements. See the NOTICE file + * distributed with this work for additional information + * regarding copyright ownership. The ASF licenses this file + * to you under the Apache License, Version 2.0 (the + * "License"); you may not use this file except in compliance + * with the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, + * software distributed under the License is distributed on an + * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY + * KIND, either express or implied. See the License for the + * specific language governing permissions and limitations + * under the License. + * + */ +#include <algorithm> + +namespace qpid { +namespace client { + +/** @file + * Private implementation of handle, include in .cpp file of handle + * subclasses _after_ including the declaration of class T. + * T can be any class that can be used with boost::intrusive_ptr. + */ + +template <class T> +Handle<T>::Handle(T* p) : impl(p) { if (impl) boost::intrusive_ptr_add_ref(impl); } + +template <class T> +Handle<T>::~Handle() { if(impl) boost::intrusive_ptr_release(impl); } + +template <class T> +Handle<T>::Handle(const Handle& h) : impl(h.impl) { if(impl) boost::intrusive_ptr_add_ref(impl); } + +template <class T> +Handle<T>& Handle<T>::operator=(const Handle<T>& h) { Handle<T>(h).swap(*this); return *this; } + +template <class T> +void Handle<T>::swap(Handle<T>& h) { std::swap(impl, h.impl); } + + +/** Access to private impl of a Handle */ +template <class T> +class HandlePrivate { + public: + static boost::intrusive_ptr<T> get(Handle<T>& h) { return boost::intrusive_ptr<T>(h.impl); } +}; + + +}} // namespace qpid::client + +#endif /*!QPID_CLIENT_HANDLEPRIVATE_H*/ |
