[Bug 57666] SXSSFWorkbook.dispose() does not delete temp xml files of sheets copied from an existing XSSFWorkbook at construction time

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=57666

--- Comment #3 from Yaniv Kunda <ya...@kundas.net> ---
I don't have a proper environment for patching right now, but I would change
SXSSFWorkbook.removeSheetAt(int) from

    public void removeSheetAt(int index)
    {
        XSSFSheet xSheet=_wb.getSheetAt(index);
        _wb.removeSheetAt(index);
        deregisterSheetMapping(xSheet);
    }

to

    public void removeSheetAt(int index)
    {
        XSSFSheet xSheet=_wb.getSheetAt(index);
        _wb.removeSheetAt(index);
        SXSSFSheet sxSheet = getSXSSFSheet(xSheet);
        sxSheet.dispose();
        deregisterSheetMapping(xSheet);
    }

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org