Remove uncessary TODOs

seanking-wx · Feb 9, 2017 · 7ae534f · 7ae534f
1 parent 24bea94
commit 7ae534f
Show file tree

Hide file tree

Showing 5 changed files with 0 additions and 12 deletions.
diff --git a/ooxml/XSSF/Streaming/AutoSizeColumnTracker.cs b/ooxml/XSSF/Streaming/AutoSizeColumnTracker.cs
@@ -221,7 +221,6 @@ private bool implicitlyTrackColumn(int column)
          */
         public bool UntrackColumns(Collection<int> columns)
         {
-            //TODO: CHeck for parity
             bool result = false;
             foreach (var col in columns)
             {
@@ -291,7 +290,6 @@ public int getBestFitColumnWidth(int column, bool useMergedCells)
                             "Either explicitly track the column or track all columns.", reason);
                 }
             }
-            //TODO: does java get remove the item from the set
             double width = maxColumnWidths[column].getMaxColumnWidth(useMergedCells);
             return (int)(256 * width);
         }
@@ -305,7 +303,6 @@ public int getBestFitColumnWidth(int column, bool useMergedCells)
          * @since 3.14beta1
          */
 
-            //TODO: fix
         public void UpdateColumnWidths(IRow row)
         {
             // track new columns
@@ -350,7 +347,6 @@ public void UpdateColumnWidths(IRow row)
 
                     if (maxColumnWidths.ContainsKey(column))
                     {
-                        //TODO: pretty sure get doesn't remove entry
                         ColumnWidthPair pair = maxColumnWidths[column];
                         UpdateColumnWidth(cell, pair);
                     }

diff --git a/ooxml/XSSF/Streaming/SXSSFEvaluationSheet.cs b/ooxml/XSSF/Streaming/SXSSFEvaluationSheet.cs
@@ -39,7 +39,6 @@ public SXSSFSheet getSXSSFSheet()
 
     public SXSSFEvaluationCell getCell(int rowIndex, int columnIndex)
         {
-            //TODO: maybe why we want a sorted dict.
             SXSSFRow row = _xs._rows[rowIndex];
             if (row == null)
             {

diff --git a/ooxml/XSSF/Streaming/SXSSFFormulaEvaluator.cs b/ooxml/XSSF/Streaming/SXSSFFormulaEvaluator.cs
@@ -30,7 +30,6 @@ namespace NPOI.XSSF.Streaming
 {
     public class SXSSFFormulaEvaluator : XSSFFormulaEvaluator
     {
-        //TODO: Readonly?
         private static POILogger logger = POILogFactory.GetLogger(typeof(SXSSFFormulaEvaluator));
 
         private IWorkbook wb;
@@ -44,7 +43,6 @@ public SXSSFFormulaEvaluator(SXSSFWorkbook workbook) : this(workbook, null, null
 
         }
 
-        //TODO: implemented new constructor
         private SXSSFFormulaEvaluator(SXSSFWorkbook workbook, WorkbookEvaluator bookEvaluator) : base(bookEvaluator)
         {
             this.wb = workbook;

diff --git a/ooxml/XSSF/Streaming/SXSSFRow.cs b/ooxml/XSSF/Streaming/SXSSFRow.cs
@@ -51,7 +51,6 @@ public bool HasCustomHeight()
             return Height != -1;
         }
 
-        //TODO; Do we lose methods here? SSXSSFCELL -> ICell
         public List<ICell> Cells
         {
             get { return _cells.Values.Select(cell => (ICell) cell).ToList(); }
@@ -222,7 +221,6 @@ public ICell GetCell(int cellnum)
             return GetCell(cellnum, policy);
         }
 
-        //TODO: Test
         public ICell GetCell(int cellnum, MissingCellPolicy policy)
         {
             CheckBounds(cellnum);
@@ -325,7 +323,6 @@ public IEnumerator<ICell> GetEnumerator()
 
             public bool MoveNext()
             {
-                //TODO: in my case, i can assume no skipped cells
                 pos += 1;
                 return _cells.ContainsKey(pos);
             }
@@ -343,7 +340,6 @@ public class CellIterator : IEnumerator<ICell>
             private int pos;
             public CellIterator(int lastCellNum, Dictionary<int, SXSSFCell> cells)
             {
-                //TODO: Should be static so I don't know if i can just pass this to here
                 maxColumn = lastCellNum; //last column PLUS ONE, SHOULD BE DERIVED from cells enum.
                 pos = 0;
                 _cells = cells;

diff --git a/ooxml/XSSF/Streaming/SXSSFSheet.cs b/ooxml/XSSF/Streaming/SXSSFSheet.cs
@@ -720,7 +720,6 @@ public void RemoveMergedRegion(int index)
             _sh.RemoveMergedRegion(index);
         }
 
-        //TODO:FixME!
         public void RemoveRow(IRow row)
         {
             throw new NotImplementedException();