Adjust ownership of BaseRequestImpl sub-classes.
Keep two weak_ptr<>.
one (internal) for tracking relations with Channel.
one (external) for tracking relations w/ user code, and send Q.
external wraps internal, and calls destroy() to break
ref. loops involving internal.
death to m_thisPointer!
Drop unnecessary "worker" thread which does no work.
Ensure that returned shared_ptr is unique()==true.
Add ServerContext::create() to start a new server
with specific config and/or providers
This will forevermore be 1, so no
reason to incur complexity of testing this.
size % PVA_ALIGNMENT -> 0
alignBuffer(PVA_ALIGNMENT) -> no-op
alignedValue(val, PVA_ALIGNMENT) -> val
Replace interface+impls with a single concrete container
which wraps a map.
This breaks code which was sub-classing.
However, no external code can be found which does this.
A build() method is used instead of simple ctor so
that the ctor is private, which will cause theoretical
external sub-class code to fail to compile.
Code using getChannelProviderRegistry()
and existing methods will behave as before.
Add ChannelProviderRegistry::getFactory() for pass through
to ChannelProviderFactory (Registery is just a proxy()...)
Remove existing get/create virtual method in favor of getFactory.
Update both existing implementations.
Add SimpleChannelProviderFactory to cut down on boilerplace
for the common case.
Add method to clear mapping and release any saved sharedInstance.
Deprecate ChannelProvider::configure(), which doesn't do much and
is incompatible with the idea of shared context.
A lot of down-stream mess related to the confused relationship
between InternalClientContextImpl and InternalClientContextImpl::ChannelProviderImpl.
This is changed to compose the provider within the context
and use a nested shared_ptr so that references to the provider
are really references to the context.
This brings the ownership semantic in line with what the API
suggests, and what other providers implement.
