This is an automated email from the ASF dual-hosted git repository.
pengzheng pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/celix.git
from 29fa7d0f Merge pull request #722 from apache/hotfix/codecov-fix
add 3e61888b Replace dfi_log_util.h by celix_err.h.
add 452bfb36 Make gcov play nicely with Conan2.
add ba397036 #509 Remove embedded memstream from utils.
add ccbd7b09 Add error injection for dyn_interface.
add ad1fa88e Remove unnecessary null check (guaranteed by class invariant).
add cced65cf Code deduplication and coverage improvements for dyn_common.
add 45bc7d2f Add more tests for dyn_common.c
add 2a0314d7 Add const qualifier to dyn_interface.h and apply early return
to dynInterface_parse.
add e27d73af Add const qualifier, apply early return error handling
patterns, and improve interface ergonomic to dyn_interface.
add 9e9b14f5 Improve ergonomics of dynInterface_methods.
add b6d5e395 Apply const qualifier to dyn_type and improve ergonomics for
functions that cannot fail.
add 20ef8891 Merge branch 'master' into feature/dfi-cleanup
add 04692bce Add dynType_parseOfName to avoid unnecessary string
duplication during parsing.
add 7936dca6 Extract common functionality of dyn_interface and dyn_message
into dyn_descriptor and improve ergonomics of dyn_message API.
add b7354954 Avoid explicit function pointer conversion.
add d0c1ede2 Extend dynType_parseWithStreamOfName for code deduplication,
and remove tail recursion from dynType_findType.
add 179a3bf8 Merge branch 'master' into feature/dfi-cleanup
add 24228794 Refactor dynType_parseComplex to return early on error and
handle error of missing the closing brace.
add 53c5da76 Remove tail recursion in dynType_alloc and add more parsing
tests.
add 0e790759 Merge branch 'master' into feature/dfi-cleanup
add 32be748a Merge branch 'master' into feature/dfi-cleanup
add 9758b649 Merge branch 'master' into feature/dfi-cleanup
add 90dd0c07 Merge branch 'master' into feature/dfi-cleanup
add 78b7d2d8 Refactor dynType_sequence_alloc and dynType_sequence_reserve.
add 84b45788 Refactor other dynType_sequence methods.
add dfc647c0 Merge branch 'master' into feature/dfi-cleanup
add e84ac213 Make double-reference work for dynType, and improve
dynType_print robustness and coverage.
add a6e22f76 Upgrade to conan 1.62.
add 1ff4bc15 Skip typedefs for reference when printing a dynType.
add 0bde38e5 Only Skip typedefs already printed when printing a dynType.
add e3562a5f Apply const qualifier to dynFunction.
add bfc57bdd Refactor dynFunction_parse and dynFunction_parseWithStr.
add 1c370b1e Add more tests for dynFunction_parse.
add 86a90957 Code cleanup and testing coverage improvement of dynFunction.
add 4c2b4d41 Treat unknown complex member as error when parsing and
refactor json deserialization.
add 4c6d4dbd More tests for `jsonSerializer_parseAny`.
add b61b6bd4 Fix unit test error in
TEST_F(JsonSerializerErrorInjectionTestSuite, SerilizationError).
add 8e38c03f Fix compilation error linux-build-apt CI.
add 58589ade Fix compilation error linux-build-apt CI.
add 3f039d3e Add option to control whether to install Find modules defined
by Celix, and skip it in conan build.
add f6a42e85 Remove unnecessary find_dependency from CelixDeps.cmake.in.
add e1dd1b88 Improve dyn_type deserialization.
add 88d43dd7 Merge branch 'master' into feature/dfi-cleanup
add 927926c8 Remove unnecessary recursions caused by type reference by
name.
add 782325e6 Improve dyn_type serialization.
add 7f1391d4 Add more serialization tests.
add 6fffb96b Support nullptr when serialize text.
add 1220311c Optimize dyn_function's arguments handling.
add 74f1243d Avoid list iteration in dynInterface_nrOfMethods.
add 6223856e Add dynType_realType for code deduplication.
add ba398d32 Remove unnecessary calloc and fix crash caused by pointer
reference in jsonRpc_call.
add 797e9755 Add some type checking to dyn_function.
add 86249373 Update codecov-action.
add f8966015 Merge branch 'master' into feature/dfi-cleanup
add e34eb00b #723 Add triviality test for dynType so that definitions of
dfi argument types can be given.
add 823c131b Fix compilation error.
add aebe9750 Merge triviality test for dynType into parsing.
add 28db6c37 Perform more strict checking on interface method.
add 56abd7d5 Optimize jsonRpc_handleReply.
add d833f70b Fix issue caused by reference type in jsonRpc_handleReply and
optimize jsonRpc_prepareInvokeRequest.
add c39b5814 Fix an issue caused by text reference in
jsonRpc_prepareInvokeRequest.
add 94903a2f Optimize jsonRpc_call.
add 4301a982 Add test for dynFunction_getName.
add bfef295a Enhance checking in dynInterface.
add e3120aef Optimize jsonRpc_call.
add 51f0f1d9 Fix compilation errors.
add 876471dd Fix stack-use-after-scope reported by clang build.
add 8d1b0bf8 Add test for dynFunction_argumentMetaForIndex.
add d099a704 Apply suggestions from code review
add ca7dab29 Fix failing test.
add 90d643a9 Some early return fixes and documentation improvements.
new ce135f54 Merge pull request #699 from apache/feature/dfi-cleanup
The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
Summary of changes:
CMakeLists.txt | 5 +-
.../src/export_registration_dfi.c | 8 +-
.../src/import_registration_dfi.c | 45 +-
.../gtest/src/RsaJsonRpcUnitTestSuite.cc | 33 -
.../rsa_rpc_json/src/rsa_json_rpc_endpoint_impl.c | 9 +-
.../rsa_rpc_json/src/rsa_json_rpc_impl.c | 1 +
.../rsa_rpc_json/src/rsa_json_rpc_proxy_impl.c | 25 +-
cmake/celix_project/CodeCoverage.cmake | 4 +-
conanfile.py | 3 +
libs/dfi/CMakeLists.txt | 12 +-
libs/dfi/error_injector/CMakeLists.txt | 3 +-
libs/dfi/error_injector/dfi/CMakeLists.txt | 1 -
libs/dfi/error_injector/dfi/include/dfi_ei.h | 2 -
libs/dfi/error_injector/dfi/src/dfi_ei.cc | 7 -
.../dfi/error_injector/ffi}/CMakeLists.txt | 17 +-
.../error_injector/ffi/include/ffi_ei.h} | 16 +-
libs/dfi/error_injector/ffi/src/ffi_ei.cc | 44 +
libs/dfi/gtest/CMakeLists.txt | 30 +-
libs/dfi/gtest/descriptors/example6.descriptor | 6 +-
.../{example1.descriptor => example7.descriptor} | 4 +-
.../gtest/descriptors/invalids/garbage.descriptor | 1 +
.../invalidExtraSection.descriptor} | 2 +
.../invalidInterfaceAnnotations.descriptor} | 2 +-
.../invalids/invalidMetaType.descriptor | 2 +-
...tor => invalidMethodMissingEquality.descriptor} | 2 +-
...=> invalidMethodMissingFunctionName.descriptor} | 2 +-
...escriptor => invalidMethodMissingId.descriptor} | 2 +-
.../invalidMsgMissingName.descriptor} | 1 -
.../invalidMsgMissingNewline.descriptor} | 2 +-
.../invalids/invalidMsgMissingVersion.descriptor | 3 +-
.../invalidMsgType.descriptor} | 2 +-
...iptor => invalidTypeMissingEquality.descriptor} | 2 +-
...> invalidTypeUnrecognizedSimpleType.descriptor} | 2 +-
.../methodMissingHandle.descriptor} | 2 +-
...riptor => methodWithMultipleHandles.descriptor} | 4 +-
...descriptor => methodWithTooManyArgs.descriptor} | 4 +-
.../methodWithoutArguments.descriptor} | 2 +-
.../descriptors/invalids/multiOutArgs.descriptor | 1 -
...tArgs.descriptor => multiPreOutArgs.descriptor} | 1 -
.../noName.descriptor} | 1 -
.../noType.descriptor} | 1 -
...nvalidType.descriptor => noTypeName.descriptor} | 2 +-
.../outArgAtWrongPosition.descriptor} | 2 +-
libs/dfi/gtest/src/dyn_closure_tests.cpp | 14 -
libs/dfi/gtest/src/dyn_common_ei_tests.cc | 84 ++
libs/dfi/gtest/src/dyn_common_tests.cc | 149 ++++
libs/dfi/gtest/src/dyn_example_functions.c | 4 +
libs/dfi/gtest/src/dyn_example_functions.h | 2 +
libs/dfi/gtest/src/dyn_function_ei_tests.cc | 112 +++
libs/dfi/gtest/src/dyn_function_tests.cpp | 184 ++--
libs/dfi/gtest/src/dyn_interface_ei_tests.cc | 101 +++
libs/dfi/gtest/src/dyn_interface_tests.cpp | 267 ++++--
.../dfi/gtest/src/dyn_message_ei_tests.cc | 41 +-
libs/dfi/gtest/src/dyn_message_tests.cpp | 84 +-
libs/dfi/gtest/src/dyn_type_ei_tests.cc | 192 ++++
libs/dfi/gtest/src/dyn_type_tests.cpp | 338 ++++++-
libs/dfi/gtest/src/json_rpc_ei_tests.cc | 189 ++++
libs/dfi/gtest/src/json_rpc_test.c | 67 ++
.../dfi/gtest/src/json_rpc_test.h | 30 +-
libs/dfi/gtest/src/json_rpc_tests.cpp | 719 +++++++--------
libs/dfi/gtest/src/json_serializer_ei_tests.cc | 172 ++++
libs/dfi/gtest/src/json_serializer_tests.cpp | 242 ++++-
libs/dfi/include/dfi_log_util.h | 78 --
libs/dfi/include/dyn_function.h | 54 +-
libs/dfi/include/dyn_interface.h | 67 +-
libs/dfi/include/dyn_message.h | 51 +-
libs/dfi/include/dyn_type.h | 128 +--
libs/dfi/include/json_rpc.h | 10 +-
libs/dfi/include/json_serializer.h | 20 +-
libs/dfi/src/dyn_common.c | 184 ++--
libs/dfi/{include => src}/dyn_common.h | 35 +-
libs/dfi/src/dyn_descriptor.c | 178 ++++
libs/dfi/src/dyn_descriptor.h | 51 ++
libs/dfi/src/dyn_function.c | 323 ++++---
libs/dfi/src/dyn_function_common.h | 23 +-
libs/dfi/src/dyn_interface.c | 459 +++-------
libs/dfi/src/dyn_interface_common.h | 6 +-
libs/dfi/src/dyn_message.c | 344 ++------
libs/dfi/src/dyn_type.c | 972 +++++++++------------
libs/dfi/src/dyn_type_common.c | 56 +-
libs/dfi/src/dyn_type_common.h | 26 +-
libs/dfi/src/json_rpc.c | 582 ++++++------
libs/dfi/src/json_serializer.c | 520 +++++------
libs/error_injector/CMakeLists.txt | 6 +
.../{ifaddrs => jansson}/CMakeLists.txt | 25 +-
.../asprintf_ei.h => jansson/include/jansson_ei.h} | 20 +-
libs/error_injector/jansson/src/jansson_ei.cc | 91 ++
libs/error_injector/stdio/CMakeLists.txt | 2 +
libs/error_injector/stdio/include/stdio_ei.h | 8 +-
libs/error_injector/stdio/src/stdio_ei.cc | 16 +
libs/error_injector/string/CMakeLists.txt | 1 +
libs/error_injector/string/include/string_ei.h | 2 +
libs/error_injector/string/src/string_ei.cc | 14 +-
libs/framework/gtest/src/ManifestTestSuite.cc | 4 -
libs/framework/src/celix_log.c | 4 -
libs/utils/CMakeLists.txt | 14 -
libs/utils/include_deprecated/celix_utils_api.h | 5 -
libs/utils/src/memstream/fmemopen.c | 76 --
libs/utils/src/memstream/open_memstream.c | 130 ---
99 files changed, 4407 insertions(+), 3489 deletions(-)
copy {bundles/logging/log_helper/gtest =>
libs/dfi/error_injector/ffi}/CMakeLists.txt (68%)
copy libs/{error_injector/sys_shm/include/sys_shm_ei.h =>
dfi/error_injector/ffi/include/ffi_ei.h} (80%)
create mode 100644 libs/dfi/error_injector/ffi/src/ffi_ei.cc
copy libs/dfi/gtest/descriptors/{example1.descriptor => example7.descriptor}
(64%)
create mode 100644 libs/dfi/gtest/descriptors/invalids/garbage.descriptor
copy libs/dfi/gtest/descriptors/{example1.descriptor =>
invalids/invalidExtraSection.descriptor} (89%)
copy libs/dfi/gtest/descriptors/{example1.descriptor =>
invalids/invalidInterfaceAnnotations.descriptor} (90%)
copy libs/dfi/gtest/descriptors/invalids/{invalidMethodReturnType.descriptor
=> invalidMethodMissingEquality.descriptor} (72%)
copy libs/dfi/gtest/descriptors/invalids/{invalidMethodReturnType.descriptor
=> invalidMethodMissingFunctionName.descriptor} (72%)
copy libs/dfi/gtest/descriptors/invalids/{invalidMethodReturnType.descriptor
=> invalidMethodMissingId.descriptor} (72%)
copy libs/dfi/gtest/descriptors/{msg_example1.descriptor =>
invalids/invalidMsgMissingName.descriptor} (94%)
copy libs/dfi/gtest/descriptors/{msg_example1.descriptor =>
invalids/invalidMsgMissingNewline.descriptor} (76%)
copy libs/dfi/gtest/descriptors/{msg_example1.descriptor =>
invalids/invalidMsgType.descriptor} (76%)
copy libs/dfi/gtest/descriptors/invalids/{invalidType.descriptor =>
invalidTypeMissingEquality.descriptor} (80%)
copy libs/dfi/gtest/descriptors/invalids/{invalidType.descriptor =>
invalidTypeUnrecognizedSimpleType.descriptor} (80%)
copy libs/dfi/gtest/descriptors/{example1.descriptor =>
invalids/methodMissingHandle.descriptor} (88%)
copy libs/dfi/gtest/descriptors/invalids/{invalidMethodReturnType.descriptor
=> methodWithMultipleHandles.descriptor} (51%)
copy libs/dfi/gtest/descriptors/invalids/{invalidType.descriptor =>
methodWithTooManyArgs.descriptor} (50%)
copy libs/dfi/gtest/descriptors/{example1.descriptor =>
invalids/methodWithoutArguments.descriptor} (88%)
copy libs/dfi/gtest/descriptors/invalids/{multiOutArgs.descriptor =>
multiPreOutArgs.descriptor} (70%)
copy libs/dfi/gtest/descriptors/{example1.descriptor =>
invalids/noName.descriptor} (95%)
copy libs/dfi/gtest/descriptors/{example1.descriptor =>
invalids/noType.descriptor} (95%)
copy libs/dfi/gtest/descriptors/invalids/{invalidType.descriptor =>
noTypeName.descriptor} (80%)
copy libs/dfi/gtest/descriptors/{example5.descriptor =>
invalids/outArgAtWrongPosition.descriptor} (57%)
create mode 100644 libs/dfi/gtest/src/dyn_common_ei_tests.cc
create mode 100644 libs/dfi/gtest/src/dyn_common_tests.cc
create mode 100644 libs/dfi/gtest/src/dyn_function_ei_tests.cc
create mode 100644 libs/dfi/gtest/src/dyn_interface_ei_tests.cc
copy examples/celix-examples/dm_example_cxx/phase1/src/Phase1Cmp.h =>
libs/dfi/gtest/src/dyn_message_ei_tests.cc (51%)
create mode 100644 libs/dfi/gtest/src/dyn_type_ei_tests.cc
create mode 100644 libs/dfi/gtest/src/json_rpc_ei_tests.cc
create mode 100644 libs/dfi/gtest/src/json_rpc_test.c
copy
bundles/remote_services/remote_service_admin_shm_v2/shm_pool/src/shm_pool_private.h
=> libs/dfi/gtest/src/json_rpc_test.h (67%)
create mode 100644 libs/dfi/gtest/src/json_serializer_ei_tests.cc
delete mode 100644 libs/dfi/include/dfi_log_util.h
rename libs/dfi/{include => src}/dyn_common.h (71%)
create mode 100644 libs/dfi/src/dyn_descriptor.c
create mode 100644 libs/dfi/src/dyn_descriptor.h
copy libs/error_injector/{ifaddrs => jansson}/CMakeLists.txt (53%)
copy libs/error_injector/{asprintf/include/asprintf_ei.h =>
jansson/include/jansson_ei.h} (65%)
create mode 100644 libs/error_injector/jansson/src/jansson_ei.cc
delete mode 100644 libs/utils/src/memstream/fmemopen.c
delete mode 100644 libs/utils/src/memstream/open_memstream.c
