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.

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

fix: remove memory leak in vector.c #1302

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

Conversation

vil02
Copy link
Member

@vil02 vil02 commented Sep 12, 2023

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
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
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
Labels
dont-close This issue/pull request shouldn't be closed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants