fixes several issues for 0.2.1 release

Author: monitorjbl

pom.xml

@@ -5,7 +5,7 @@
   <modelVersion>4.0.0</modelVersion>
   <groupId>com.thundermoose</groupId>
   <artifactId>xlsx-streamer</artifactId>
-  <version>0.3-SNAPSHOT</version>
+  <version>0.2.1</version>
 
   <build>
     <plugins>
@@ -23,6 +23,19 @@
         <artifactId>maven-install-plugin</artifactId>
         <version>2.5.1</version>
       </plugin>
+      <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
+        <artifactId>maven-source-plugin</artifactId>
+        <version>2.2.1</version>
+        <executions>
+          <execution>
+            <id>attach-sources</id>
+            <goals>
+              <goal>jar</goal>
+            </goals>
+          </execution>
+        </executions>
+      </plugin>
     </plugins>
   </build>
 

src/main/java/com/thundermoose/xlsx/StreamingReader.java

@@ -127,7 +127,7 @@ private void handleEvent(XMLEvent event) throws SAXException {
 
       if (endElement.getName().getLocalPart().equals("v")) {
         currentCell.setContents(lastContents);
-        currentRow.getCellList().add(currentCell);
+        currentRow.getCellMap().put(currentCell.getColumnIndex(), currentCell);
       }
 
     }

src/main/java/com/thundermoose/xlsx/impl/StreamingRow.java

@@ -7,26 +7,26 @@
 import org.apache.poi.ss.usermodel.Sheet;
 
 import java.util.Iterator;
-import java.util.LinkedList;
-import java.util.List;
+import java.util.Map;
+import java.util.TreeMap;
 
 public class StreamingRow implements Row {
   private int rowIndex;
-  private List<Cell> cellList = new LinkedList<Cell>();
+  private Map<Integer, Cell> cellMap = new TreeMap<>();
 
   public StreamingRow(int rowIndex) {
     this.rowIndex = rowIndex;
   }
 
-  public List<Cell> getCellList() {
-    return cellList;
+  public Map<Integer, Cell> getCellMap() {
+    return cellMap;
   }
 
-  public void setCellList(List<Cell> cellList) {
-    this.cellList = cellList;
+  public void setCellMap(Map<Integer, Cell> cellMap) {
+    this.cellMap = cellMap;
   }
 
-  /* Supported */
+ /* Supported */
 
   @Override
   public int getRowNum() {
@@ -35,17 +35,17 @@ public int getRowNum() {
 
   @Override
   public Iterator<Cell> cellIterator() {
-    return cellList.iterator();
+    return cellMap.values().iterator();
   }
 
   @Override
   public Iterator<Cell> iterator() {
-    return cellList.iterator();
+    return cellMap.values().iterator();
   }
 
   @Override
   public Cell getCell(int cellnum) {
-    return cellList.size() > cellnum ? cellList.get(cellnum) : null;
+    return cellMap.get(cellnum);
   }
 
   /* Not supported */