Closed iraj720 closed 5 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 99.21%. Comparing base (
5f8a5b8
) to head (e098c3a
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
PR Details
fix Get Rows function memory issues that allocate huge amount of memory in special xlsx files
Description
the problem is discussed in the issue here i just changed it in a way that those empty rows that are located at the end of rows will be ignored
["A", "B", nil, "C", nil, nil] => ["A", "B", nil, "C"]
these nils exist in some special files more than normal for me was 1 million nils
Related Issue
https://github.com/qax-os/excelize/issues/1874
Motivation and Context
it was allocating 30Mb of memory for a 10kb file
How Has This Been Tested
discussed in the issue
Types of changes
Checklist