Skip to content

Comments

Test memory leak in resetMessages()#79

Open
uh-zuh wants to merge 1 commit intoCarbonPHP:masterfrom
uh-zuh:master
Open

Test memory leak in resetMessages()#79
uh-zuh wants to merge 1 commit intoCarbonPHP:masterfrom
uh-zuh:master

Conversation

@uh-zuh
Copy link

@uh-zuh uh-zuh commented Dec 9, 2025

Add test for #57

New test fails because "B is roughly twice A".

Is the test good enough?

Additional:

  • phpunit has no --verbose parameter since version 10

@kylekatarnls
Copy link
Contributor

Hi, just a note to say that I didn't give up on this. memory_get_usage() don't show the same behavior on my locale machine. Also inspecting the private properties of the translator, I cannot see any difference between what is stored after the first call versus the second one.

So I'm still trying to find what in Carbon or Symfony could pile up in memory.

@uh-zuh
Copy link
Author

uh-zuh commented Jan 16, 2026

Thank you! I'm still waiting :)

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.

2 participants