Skip to content

Remove C bindings - #24

Merged
jacnils merged 2 commits into
masterfrom
removed-c-bindings
Aug 2, 2026
Merged

jacnils merged 2 commits into
masterfrom
removed-c-bindings

Conversation

@jacnils

@jacnils jacnils commented Aug 2, 2026

Copy link
Copy Markdown
Owner

Rationale:

These bindings are horrendously out of date and were never even close to achieving feature-parity. Frankly put, if you were able to utilize these in their previous state, you might as well use socket APIs directly, since the only thing this abstracted away was the cross-platform aspect of it all.

Whilst I intend on writing new bindings for netkit, they will most likely be in the form of a separate project, as to not hinder progress here. C bindings are not, and have never been a priority, so it seems redundant to keep known broken code around.

Rationale:

These bindings are horrendously out of date and were never even close to achieving feature-parity. Frankly put, if you were able to utilize these in their previous state, you might as well use socket APIs directly, since the only thing this abstracted away was the cross-platform aspect of it all.

Whilst I intend on writing new bindings for netkit, they will most likely be in the form of a separate project, as to not hinder progress here. C bindings are not, and have never been a priority, so it seems redundant to keep known broken code around.
@jacnils
jacnils marked this pull request as ready for review August 2, 2026 03:23
@jacnils
jacnils merged commit e354e85 into master Aug 2, 2026
5 of 6 checks passed
@jacnils
jacnils deleted the removed-c-bindings branch August 2, 2026 03:27
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