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

fixed memory leaks #3

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

Conversation

winterrdog
Copy link

@winterrdog winterrdog commented Nov 21, 2022

I fixed the memory that was being leaked by the file name, rows, characters and highlights pointers before the program exits. Therefore this C program is 100% leak-free( for now 💯 :) ). U can take a look at the commits, review them and maybe try verifying it for yourself in GDB or Valgrind.

Thanks a lot for this project @antirez ( U might have stopped writing code :( , but I'm glad u shaped my programming life )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant