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

Try to fix memory leaks #3

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

Try to fix memory leaks #3

wants to merge 8 commits into from

Conversation

Vianpyro
Copy link
Owner

@Vianpyro Vianpyro commented Sep 11, 2024

Before: 😰

LEAK SUMMARY:
   definitely lost: 9,373,704 bytes in 250,699 blocks
   indirectly lost: 6,932,840 bytes in 206,429 blocks
     possibly lost: 40 bytes in 1 blocks
   still reachable: 0 bytes in 0 blocks
        suppressed: 0 bytes in 0 blocks

Now: 🥴

LEAK SUMMARY:
   definitely lost: 981,384 bytes in 40,891 blocks
   indirectly lost: 6,931,512 bytes in 206,387 blocks
     possibly lost: 1,368 bytes in 43 blocks
   still reachable: 0 bytes in 0 blocks
        suppressed: 0 bytes in 0 blocks

Command to run Valgrind:

gcc ./*.c ./solve/*.c -o 8-puzzle && valgrind --leak-check=full --show-leak-kinds=all --track-origins=yes --verbose --log-file=valgrind-out.txt ./8-puzzle -p 386257041

valgrind-out.txt

@Vianpyro Vianpyro added the enhancement New feature or request label Sep 11, 2024
@Vianpyro Vianpyro self-assigned this Sep 11, 2024
@Vianpyro Vianpyro added the help wanted Extra attention is needed label Sep 11, 2024
@Vianpyro Vianpyro changed the title Tried to fix memory leaks Try to fix memory leaks Sep 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant