Skip to content

[C API] Move C API samples to the examples directory - #367

Merged
rfsaliev merged 1 commit into
dev/c-apifrom
rfsaliev/c-api-move-examples2
Aug 19, 2026
Merged

[C API] Move C API samples to the examples directory#367
rfsaliev merged 1 commit into
dev/c-apifrom
rfsaliev/c-api-move-examples2

Conversation

@rfsaliev

Copy link
Copy Markdown
Member

No description provided.

@rfsaliev
rfsaliev marked this pull request as ready for review August 19, 2026 15:04
@rfsaliev
rfsaliev merged commit b6d057b into dev/c-api Aug 19, 2026
22 checks passed
@rfsaliev
rfsaliev deleted the rfsaliev/c-api-move-examples2 branch August 19, 2026 15:05
rfsaliev added a commit that referenced this pull request Aug 24, 2026
**Important note:**
> **This API refactoring breaks compatibility with existing client code**

Refactor API for better consitency, stability, extensibility.
- Updated ThreadPoolBuilder to ensure custom threadpool pointers are validated and initialized correctly.
- Enhanced error handling in parallel_for method to catch exceptions and rethrow them appropriately.
- Modified IDFilterAdapter to check for null operations and validate filter rates during initialization.
- Adjusted test cases to reflect changes in function signatures and ensure proper error handling.
- Introduced new utility functions for initializing search results and memory breakdown structures.
- Updated sequential threadpool implementation to return a boolean indicating success
- All public headers moved to `include/svs/c/`, and installation paths
updated to match, replacing the old `c_api` directory.
- Added generated version header `svs_c_version.h` with version macros, configured and installed via CMake.
- Refactored `svs_search_result_t` structure now allows user to pre-allocate result buffers.
- Added a detailed `README.md` for the C API, including build instructions, usage, and sample code.
- Moved C API samples to the examples directory (#367)
- Addressed API contract issues and resolve implementation bugs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant