Skip to content

Add disjoint-set data structure#136

Merged
tezc merged 4 commits into
tezc:masterfrom
Ayilay:disjoint_set
May 18, 2026
Merged

Add disjoint-set data structure#136
tezc merged 4 commits into
tezc:masterfrom
Ayilay:disjoint_set

Conversation

@Ayilay

@Ayilay Ayilay commented Dec 12, 2025

Copy link
Copy Markdown
Contributor

Contains README, and Unit Tests using standard asserts.

Contains additional unit tests using Unity (not included). The file unity_test_disjoint.c is ignored by the build system, but serves as reference for additional valid use cases.

Code style is mimicked to the best of my ability to match the other data structures in this collection.

I needed to use a disjoint set for day8 of Advent of Code, which I prefer to do in C. Hope it makes a welcome addition to your collection.

@Ayilay

Ayilay commented May 16, 2026

Copy link
Copy Markdown
Contributor Author

Kindly checking in a few months later to see if this feature is still interesting. I'd be happy to make the necessary adjustments to help this get merged. Kind regards.

@tezc

tezc commented May 16, 2026

Copy link
Copy Markdown
Owner

Hi @Ayilay sorry, I was swamped with other things. I'll review this shortly. Thank you for your patience.

@tezc tezc closed this May 16, 2026
@tezc tezc reopened this May 16, 2026
@tezc

tezc commented May 16, 2026

Copy link
Copy Markdown
Owner

@Ayilay Please see the CI failure above. Could you copy a CMakeLists.txt from another directory to disjoint dir?
For example, you can copy this one and rename uri to disjoint in a few lines.: https://github.com/tezc/sc/blob/master/uri/CMakeLists.txt

Comment thread disjoint/unity_test_disjoint.c Outdated
Ayilay added 3 commits May 18, 2026 07:23
Contains README, and Unit Tests using standard asserts.

Contains additional unit tests using Unity (not included).
The file unity_test_disjoint.c is ignored by the build system,
but serves as reference for additional valid use cases.

Code style is mimiced to the best of my ability to match the
other data structures in this collection.
Comment thread disjoint/CMakeLists.txt Outdated
@tezc tezc merged commit c2152a3 into tezc:master May 18, 2026
26 of 27 checks passed
@tezc

tezc commented May 18, 2026

Copy link
Copy Markdown
Owner

@Ayilay thank you for this one!

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.

2 participants