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

feat: custom memory management lib #81

Merged
merged 1 commit into from
Jan 18, 2025
Merged

feat: custom memory management lib #81

merged 1 commit into from
Jan 18, 2025

Conversation

araujo88
Copy link
Member

@araujo88 araujo88 commented Jan 18, 2025 •
edited
Loading

Description

  • refactored hashlib to include documentation annotations
  • introduced custom memory management lib to replace custom memory allocation calls

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Performance improvement

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have documented my changes in the code or documentation
  • I have added tests that prove my changes work (if applicable)
  • All new and existing tests pass

@araujo88 araujo88 added enhancement New feature or request refactor Modify existing working code but keeping the same functionality. labels Jan 18, 2025
@araujo88 araujo88 force-pushed the feat/custom-memory-management-lib branch from 77263b2 to 817bc69 Compare January 18, 2025 19:49
@SIGMazer
Copy link
Contributor

Nice

@SIGMazer SIGMazer self-requested a review January 18, 2025 19:55
@SIGMazer SIGMazer merged commit e446812 into main Jan 18, 2025
2 checks passed
@SIGMazer SIGMazer deleted the feat/custom-memory-management-lib branch January 18, 2025 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SIGMazer SIGMazer SIGMazer approved these changes

@PatriciaLisboa PatriciaLisboa Awaiting requested review from PatriciaLisboa

@pamgaiguer pamgaiguer Awaiting requested review from pamgaiguer

@AyuBloom AyuBloom Awaiting requested review from AyuBloom

@VeryElegantBread VeryElegantBread Awaiting requested review from VeryElegantBread

Assignees
No one assigned
Labels
enhancement New feature or request refactor Modify existing working code but keeping the same functionality.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@araujo88 @SIGMazer