pnoltes opened a new pull request, #422:
URL: https://github.com/apache/celix/pull/422
Adds an alternative `celix_string_hash_map_t` and `celix_long_hash_map_t`
implementation for to `hash_map_t`
All Celix C function shoulds be prefixed with `celix_` , functions without a
`celix_` prefix are deprecated and will be removed in the future.
I think this PR adds the last missing `celix_` api so that all non `celix_`
prefixed function have an alternative api.
The PR introduces 2 versions of a C hash map, 1 which uses a char* as key
and one that uses a long as key. In my experience most - if not all - use cases
of a hash map can be achieved with a string or long (intptr_t, bundle id,
service id, etc) key.
How hash map iterators work is also updated so that this is more in-line
with C++.
Note that this PR only introduces an alternative hash map impl, but does not
yet introduce the usage of the alternative hash map. This will be done in an
separate PR.
This PR also contains some other changes:
- Adds missing getManifestValue call to the C and C++ bundle api.
- Updates celix array list to also support configuring removed callbacks
- Adds some celix aray list gtest next to the existing Cpputest.
- Updates the utils benchmarks so that the new string and long hash map are
also benched marked. And adds a benchmark for filling hash maps.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@celix.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
