-
Notifications
You must be signed in to change notification settings - Fork 2.1k
RFC: Merge utlist with clist/list #6209
Copy link
Copy link
Open
Labels
Discussion: RFCThe issue/PR is used as a discussion starting point about the item of the issue/PRThe issue/PR is used as a discussion starting point about the item of the issue/PRState: don't staleState: Tell state-bot to ignore this issueState: Tell state-bot to ignore this issueType: cleanupThe issue proposes a clean-up / The PR cleans-up parts of the codebase / documentationThe issue proposes a clean-up / The PR cleans-up parts of the codebase / documentationType: enhancementThe issue suggests enhanceable parts / The PR enhances parts of the codebase / documentationThe issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation
Description
Currently we have two separate linked list implementations:
clist/list: our native list implememations incoreimplementing singly-linked linear (list) and circular lists (clist).utlist: imported linked-list implementation insysimplementing singly-linked linear lists (LL_and doubly-linked linear (DL_) and circular lists (CDL_).
Both implementations have their respective benefits and drawbacks:
| Implementation | Benefits | Drawbacks |
|---|---|---|
| core |
|
|
| utlist |
|
|
I see benefits in keeping both, but the risk of code duplication exists (and in case of singly-linked lists is already present).
Reactions are currently unavailable
Metadata
Metadata
Labels
Discussion: RFCThe issue/PR is used as a discussion starting point about the item of the issue/PRThe issue/PR is used as a discussion starting point about the item of the issue/PRState: don't staleState: Tell state-bot to ignore this issueState: Tell state-bot to ignore this issueType: cleanupThe issue proposes a clean-up / The PR cleans-up parts of the codebase / documentationThe issue proposes a clean-up / The PR cleans-up parts of the codebase / documentationType: enhancementThe issue suggests enhanceable parts / The PR enhances parts of the codebase / documentationThe issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation