Skip to content

feat: Add C bindings for the client FDv2 configuration - #625

Draft
beekld wants to merge 1 commit into
bklimt/SDK-3037/client-fdv2-public-apifrom
bklimt/SDK-3038/client-fdv2-c-bindings
Draft

beekld wants to merge 1 commit into
bklimt/SDK-3037/client-fdv2-public-apifrom
bklimt/SDK-3038/client-fdv2-c-bindings

Conversation

@beekld

@beekld beekld commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds C bindings for the client's FDv2 configuration, so the C API and the language bindings built on it can select and configure FDv2.

  • Wraps the FDv2 configuration builder in the C binding layer, following the existing conventions (opaque handles, sub-builders freed on attach, asserting setters).
  • Adds a C-only smoke-compile test that includes each new header from a .c file, guarding against C++-only syntax leaking into a public C header.

@beekld
beekld force-pushed the bklimt/SDK-3038/client-fdv2-c-bindings branch from b6fe8cb to 5c4c69e Compare October 3, 2026 01:21
@beekld
beekld force-pushed the bklimt/SDK-3038/client-fdv2-c-bindings branch from 5c4c69e to 632adeb Compare October 5, 2026 18:00
@beekld
beekld added this pull request to stack #627 October 5, 2026 18:00
@beekld
beekld removed this pull request from stack #627 October 5, 2026 18:05
@beekld
beekld added this pull request to stack #628 October 5, 2026 18:10
@beekld
beekld force-pushed the bklimt/SDK-3038/client-fdv2-c-bindings branch from 632adeb to 5f03be0 Compare October 5, 2026 18:58
@beekld
beekld removed this pull request from stack #628 October 5, 2026 19:18
@beekld
beekld added this pull request to stack #629 October 5, 2026 19:24
@beekld
beekld force-pushed the bklimt/SDK-3038/client-fdv2-c-bindings branch 2 times, most recently from 5cfe319 to 8b0373a Compare October 6, 2026 05:04
@beekld
beekld force-pushed the bklimt/SDK-3038/client-fdv2-c-bindings branch from 8b0373a to 0fbdfc5 Compare October 6, 2026 17:22

This branch has not been deployed

No deployments
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