https://bz.apache.org/bugzilla/show_bug.cgi?id=68230
Bug ID: 68230 Summary: A region overlap issue in RowShifter Product: POI Version: 5.3.x-dev Hardware: PC Status: NEW Severity: normal Priority: P2 Component: SS Common Assignee: dev@poi.apache.org Reporter: zhong...@pku.org.cn Target Milestone: --- NPOI fixed a region overlap issue: https://github.com/nissl-lab/npoi/commit/6719bff7e54ce724d07f57de4a7e19ddc36c0a07 The buggy code is still in the trunk of POI. For example, POI fixed the following method: public List<CellRangeAddress> ShiftMergedRegions(int startRow, int endRow, int n {... if (removalNeeded(merged, startRow, endRow, n )) ... } The fixed code is as follows: public List<CellRangeAddress> ShiftMergedRegions(int startRow, int endRow, int n {... var lastCol=sheet.GetRow(startRow) != null ? sheet.GetRow(startRow).LastCellNum : sheet.GetRow(endRow) != null ? sheet.GetRow(endRow).LastCellNum : 0; if (removalNeeded(merged, startRow, endRow, n, lastCol )) ... } In the trunk, the POI code is as follows: public List<CellRangeAddress> shiftMergedRegions(int startRow, int endRow, int n) {... if(removalNeeded(merged, startRow, endRow, n)) { ...} It is quite similar to the buggy NPOI code. There are also other modifications. POI can have a similar bug when handling overlapped regions. -- 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