-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48891/
-----------------------------------------------------------
Review request for qpid, Andrew Stitcher, Cliff Jansen, and Justin Ross.
Bugs: proton-1237
https://issues.apache.org/jira/browse/proton-1237
Repository: qpid-proton-git
Description
-------
- Ported C++ connection_engine core (excluding container/event_loop stuff) to C
- Replaced the guts of the C++ connection_engine with the C engine.
PROTON-1237: c++: clean up connection_engine interface.
- make container/event_loop optional for a connection_engine.
- untangled link_namer from connection engine.
Diffs
-----
examples/cpp/mt/epoll_container.cpp 4dd15251cbc2101b347613fc6a0294f9b7f9b514
proton-c/CMakeLists.txt 65dd94da84b7cf7d63b0e382fa1d8e9199b99d34
proton-c/bindings/cpp/CMakeLists.txt 18344f4ee3469883740b61feec64edbe601447ac
proton-c/bindings/cpp/include/proton/io/connection_engine.hpp
32db158c668638c387dbf72a348a515da5661c53
proton-c/bindings/cpp/include/proton/io/container_impl_base.hpp
695125d08420b60802d34f0ef1b598b5c6a1984f
proton-c/bindings/cpp/include/proton/io/link_namer.hpp
281503b906197fd402dad30ea683a16122f6ee6c
proton-c/bindings/cpp/include/proton/thread_safe.hpp
eb4df4859f7821ab8150474b5175a61bb8d07dc2
proton-c/bindings/cpp/src/container_impl.hpp
9ffb7bb20818b465c01f4ef7a40e52c3e1d8d176
proton-c/bindings/cpp/src/container_impl.cpp
7496dc40b9a60f6add6334a7e84f1aca1bc369f0
proton-c/bindings/cpp/src/engine_test.cpp
418943550e94935d270fb573a2509c4ca26263b1
proton-c/bindings/cpp/src/id_generator.hpp
31261db9d324b6b2d1b85f9a3acded155d5dbb1c
proton-c/bindings/cpp/src/id_generator.cpp
fde16586033f57a12201b7cbca86520975db3b9c
proton-c/bindings/cpp/src/io/connection_engine.cpp
706108bb5fc7456f8bc2aa189e95a5da64f2feda
proton-c/bindings/cpp/src/proton_event.hpp
a551435e8025d153b31c16deb56b53d448aba54d
proton-c/bindings/cpp/src/thread_safe_test.cpp
0eb21b229a9100e03109be0490a2d576208ecd0d
proton-c/include/proton/connection_engine.h PRE-CREATION
proton-c/src/engine/connection_engine.c PRE-CREATION
Diff: https://reviews.apache.org/r/48891/diff/
Testing
-------
ctest -R cpp
g++, clang++ X 03, 11
The new C engine is now the implementation for the C++ engine so the existing
tests are exercising it.
Thanks,
Alan Conway
