Fix/dynamic buffer allocation#2095
Conversation
|
This PR fixes the static buffer issue in Fixes #2091 Please let me know if you’d like any changes or improvements. Thanks |
- Converted static cbbuffer array to dynamically allocated memory - Added malloc with lazy initialization on first function call - Added proper error handling for allocation failures - Added debug logging for buffer allocation - Improves memory efficiency by only allocating when needed Fixes CCExtractor#2091
CCExtractor CI platform finished running the test files on linux. Below is a summary of the test results, when compared to test for commit 032cd1c...:
Your PR breaks these cases:
Congratulations: Merging this PR would fix the following tests:
It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you). Check the result page for more info. |
CCExtractor CI platform finished running the test files on windows. Below is a summary of the test results, when compared to test for commit 032cd1c...:
Your PR breaks these cases:
Congratulations: Merging this PR would fix the following tests:
It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you). Check the result page for more info. |
In raising this pull request, I confirm the following (please check boxes):
My familiarity with the project is as follows (check one):
{pull request content here}