From: Javen O'Neal <onealj@apache.org>
Date: Fri, 27 Nov 2015 11:30:18 +0000 (+0000)
Subject: +FIXME
X-Git-Tag: REL_3_14_BETA1~71
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=229a54a2489ef4649f34d7cc148d624817b6e063;p=poi.git

+FIXME

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1716838 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/src/java/org/apache/poi/ss/util/SheetUtil.java b/src/java/org/apache/poi/ss/util/SheetUtil.java
index 184c3a5c1f..5ca5b39694 100644
--- a/src/java/org/apache/poi/ss/util/SheetUtil.java
+++ b/src/java/org/apache/poi/ss/util/SheetUtil.java
@@ -103,6 +103,8 @@ public class SheetUtil {
         Row row = cell.getRow();
         int column = cell.getColumnIndex();
 
+        // FIXME: this looks very similar to getCellWithMerges below. Consider consolidating.
+        // We should only be checking merged regions if useMergedCells is true. Why are we doing this for-loop?
         int colspan = 1;
         for (CellRangeAddress region : sheet.getMergedRegions()) {
             if (containsCell(region, row.getRowNum(), column)) {