Memory leak in \Carbon\AbstractTranslator::resetMessages()

[uh-zuh]

Describe the bug
Memory leak in \Carbon\AbstractTranslator::resetMessages()

To Reproduce

for ($i = 0; $i < 20000; $i++) {
    \Carbon\Carbon::getTranslator()->resetMessages('en');
    if (mt_rand(0, 99) === 0) {
        echo "memory: " . (int)(memory_get_usage() / 1024 / 1024) .  "\n";
    }
}

Expected behavior
No memory leak

Actual behavior
Output:

memory: 8
memory: 8
memory: 12
...
memory: 123
memory: 124
memory: 126
PHP Fatal error:  Allowed memory size of 134217728 bytes exhausted (tried to allocate 32768 bytes) in /app/laravel-app/vendor/nesbot/carbon/src/Carbon/Lang/en.php on line 1

Versions

• PHP: 8.4.11
• Carbon: 3.10.2

Additional context
I have faced with this memory leak in Pest tests in Laravel application.

[kylekatarnls]

Thanks for the reproducible code snippet. I confirm the bug and am starting investigation.

[kylekatarnls]

In the meantime, you can use \Carbon\Carbon::getTranslator()->resetMessages(); which reset all the messages and does not leak.

[uh-zuh]

I can't use this workaround because this code is called deep inside Laravel framework.

[kylekatarnls]

Fix released in 3.10.3

[uh-zuh]

@kylekatarnls

Becomes much better but small memory leak is still present.

Just increase number of iterations:

for ($i = 0; $i < 300000; $i++) {
    \Carbon\Carbon::getTranslator()->resetMessages('en');
    if (mt_rand(0, 999) === 0) {
        echo "memory: " . (int)(memory_get_usage() / 1024 / 1024) .  "\n";
    }
}

I think it is essential for long running scripts.

Could you reopen the issue?