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

fix: remove memory leak in vector.c #1302

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix: remove memory leak in vector.c #1302

wants to merge 1 commit into from

Conversation

vil02
Copy link
Member

@vil02 vil02 commented Sep 12, 2023 •
edited
Loading

Description of Change

There is a memory leak in vector.c. This PR fixes it (and removes some trailing spaces). The Awesome CI Workflow passes on my end.

References

Checklist

  • Added description of change
  • PR title follows semantic commit guidelines
  • Search previous suggestions before making a new one, as yours may be a duplicate.
  • I acknowledge that all my contributions will be made under the project's license.

Notes:
Removes memory leak in vector.c.

@vil02 vil02 marked this pull request as ready for review September 12, 2023 17:24
tjgurwara99
tjgurwara99 approved these changes Sep 13, 2023
View reviewed changes
data_structures/vector.c Show resolved Hide resolved
data_structures/vector.c Show resolved Hide resolved
@vil02 vil02 force-pushed the memory_leak_in_vector_patch branch from d182608 to ecd6993 Compare October 2, 2023 11:52
@github-actions GitHub Actions
Copy link
Contributor

This pull request has been automatically marked as abandoned because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the Stale label Nov 22, 2023
@vil02
Copy link
Member Author

vil02 commented Nov 22, 2023

@Panquesito7, @tjgurwara99, @alexpantyukhin could some of you please add a dont-close or approved label?

@github-actions github-actions bot removed the Stale label Nov 23, 2023
@tjgurwara99 tjgurwara99 added the dont-close This issue/pull request shouldn't be closed label Nov 27, 2023
@vil02
Copy link
Member Author

vil02 commented Sep 12, 2024

Happy birthday #1302! I wish you that you will get lots of accepting reviews and that you will be merged some day!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@realstealthninja realstealthninja realstealthninja approved these changes

@tjgurwara99 tjgurwara99 tjgurwara99 approved these changes

@Panquesito7 Panquesito7 Awaiting requested review from Panquesito7 Panquesito7 is a code owner

@alexpantyukhin alexpantyukhin Awaiting requested review from alexpantyukhin alexpantyukhin is a code owner

Assignees
No one assigned
Labels
dont-close This issue/pull request shouldn't be closed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@vil02 @tjgurwara99 @realstealthninja