Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Create an implementation of a generic map (#13) #31

Open
wants to merge 21 commits into
base: main
Choose a base branch
from
Open

Create an implementation of a generic map (#13) #31

wants to merge 21 commits into from

Conversation

Altanis
Copy link
Contributor

@Altanis Altanis commented Dec 26, 2022

  • Design an implementation for a generic map
  • Implement map_init
  • Implement map_free
  • Implement map_put
  • Implement map_get
    Implement map_contains Obsolete, use map_get(key) != NULL

@jakejack13 jakejack13 self-assigned this Dec 26, 2022
@jakejack13 jakejack13 added the enhancement New feature or request label Dec 26, 2022
@jakejack13 jakejack13 linked an issue Dec 26, 2022 that may be closed by this pull request
Create an implementation of a generic map #13
Closed
8 tasks
farhannaqib
farhannaqib reviewed Dec 26, 2022
View reviewed changes
src/lib/map.c Outdated Show resolved Hide resolved
@Altanis
Copy link
Contributor Author

Altanis commented Dec 26, 2022

This requires memcmp to be implemented.

jakejack13
jakejack13 requested changes Jan 2, 2023
View reviewed changes
include/lib/map.h Outdated Show resolved Hide resolved
include/lib/map.h Outdated Show resolved Hide resolved
include/lib/map.h Show resolved Hide resolved
include/lib/map.h Outdated Show resolved Hide resolved
include/stdlib/stdtypes.h Show resolved Hide resolved
src/lib/map.c Show resolved Hide resolved
src/lib/map.c Show resolved Hide resolved
include/lib/map.h Outdated Show resolved Hide resolved
src/lib/map.c Outdated Show resolved Hide resolved
include/lib/map.h Outdated Show resolved Hide resolved
@jakejack13 jakejack13 removed their assignment Mar 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@farhannaqib farhannaqib farhannaqib left review comments

@jakejack13 jakejack13 jakejack13 requested changes

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Create an implementation of a generic map
3 participants
@Altanis @jakejack13 @farhannaqib