Author: nick
Date: Sun Jan  3 20:56:40 2010
New Revision: 895477

URL: http://svn.apache.org/viewvc?rev=895477&view=rev
Log:
Add in a few bits of Generics to avoid warnings

Modified:
    poi/trunk/src/java/org/apache/poi/poifs/filesystem/DirectoryEntry.java
    poi/trunk/src/java/org/apache/poi/poifs/filesystem/DirectoryNode.java
    poi/trunk/src/ooxml/java/org/apache/poi/extractor/ExtractorFactory.java

Modified: poi/trunk/src/java/org/apache/poi/poifs/filesystem/DirectoryEntry.java
URL: 
http://svn.apache.org/viewvc/poi/trunk/src/java/org/apache/poi/poifs/filesystem/DirectoryEntry.java?rev=895477&r1=895476&r2=895477&view=diff
==============================================================================
--- poi/trunk/src/java/org/apache/poi/poifs/filesystem/DirectoryEntry.java 
(original)
+++ poi/trunk/src/java/org/apache/poi/poifs/filesystem/DirectoryEntry.java Sun 
Jan  3 20:56:40 2010
@@ -47,7 +47,7 @@
      *         implementations of Entry.
      */
 
-    public Iterator getEntries();
+    public Iterator<Entry> getEntries();
 
     /**
      * is this DirectoryEntry empty?

Modified: poi/trunk/src/java/org/apache/poi/poifs/filesystem/DirectoryNode.java
URL: 
http://svn.apache.org/viewvc/poi/trunk/src/java/org/apache/poi/poifs/filesystem/DirectoryNode.java?rev=895477&r1=895476&r2=895477&view=diff
==============================================================================
--- poi/trunk/src/java/org/apache/poi/poifs/filesystem/DirectoryNode.java 
(original)
+++ poi/trunk/src/java/org/apache/poi/poifs/filesystem/DirectoryNode.java Sun 
Jan  3 20:56:40 2010
@@ -41,7 +41,7 @@
 {
 
     // Map of Entry instances, keyed by their names
-    private Map               _entries;
+    private Map<String,Entry> _entries;
 
     // the POIFSFileSystem we belong to
     private POIFSFileSystem   _filesystem;
@@ -75,12 +75,12 @@
             });
         }
         _filesystem = filesystem;
-        _entries    = new HashMap();
-        Iterator iter = property.getChildren();
+        _entries    = new HashMap<String, Entry>();
+        Iterator<Property> iter = property.getChildren();
 
         while (iter.hasNext())
         {
-            Property child     = ( Property ) iter.next();
+            Property child     = iter.next();
             Entry    childNode = null;
 
             if (child.isDirectory())
@@ -215,7 +215,7 @@
      *         implementations of Entry.
      */
 
-    public Iterator getEntries()
+    public Iterator<Entry> getEntries()
     {
         return _entries.values().iterator();
     }
@@ -263,7 +263,7 @@
 
         if (name != null)
         {
-            rval = ( Entry ) _entries.get(name);
+            rval = _entries.get(name);
         }
         if (rval == null)
         {
@@ -416,8 +416,9 @@
         List components = new ArrayList();
 
         components.add(getProperty());
-        SortedMap sortedEntries = new TreeMap(_entries);
-        Iterator  iter          = sortedEntries.values().iterator();
+        SortedMap<String,Entry> sortedEntries = 
+               new TreeMap<String,Entry>(_entries);
+        Iterator<Entry> iter = sortedEntries.values().iterator();
 
         while (iter.hasNext())
         {

Modified: 
poi/trunk/src/ooxml/java/org/apache/poi/extractor/ExtractorFactory.java
URL: 
http://svn.apache.org/viewvc/poi/trunk/src/ooxml/java/org/apache/poi/extractor/ExtractorFactory.java?rev=895477&r1=895476&r2=895477&view=diff
==============================================================================
--- poi/trunk/src/ooxml/java/org/apache/poi/extractor/ExtractorFactory.java 
(original)
+++ poi/trunk/src/ooxml/java/org/apache/poi/extractor/ExtractorFactory.java Sun 
Jan  3 20:56:40 2010
@@ -123,8 +123,8 @@
        public static POIOLE2TextExtractor createExtractor(DirectoryNode 
poifsDir, POIFSFileSystem fs) throws IOException {
                // Look for certain entries in the stream, to figure it
                //  out from
-               for(Iterator entries = poifsDir.getEntries(); 
entries.hasNext(); ) {
-                       Entry entry = (Entry)entries.next();
+               for(Iterator<Entry> entries = poifsDir.getEntries(); 
entries.hasNext(); ) {
+                       Entry entry = entries.next();
                        
                        if(entry.getName().equals("Workbook")) {
                                return new ExcelExtractor(poifsDir, fs);
@@ -160,9 +160,9 @@
                
                if(ext instanceof ExcelExtractor) {
                        // These are in MBD... under the root
-                       Iterator it = fs.getRoot().getEntries();
+                       Iterator<Entry> it = fs.getRoot().getEntries();
                        while(it.hasNext()) {
-                               Entry entry = (Entry)it.next();
+                               Entry entry = it.next();
                                if(entry.getName().startsWith("MBD")) {
                                        dirs.add(entry);
                                }
@@ -172,9 +172,9 @@
                        try {
                                DirectoryEntry op = (DirectoryEntry)
                                        fs.getRoot().getEntry("ObjectPool");
-                               Iterator it = op.getEntries();
+                               Iterator<Entry> it = op.getEntries();
                                while(it.hasNext()) {
-                                       Entry entry = (Entry)it.next();
+                                       Entry entry = it.next();
                                        if(entry.getName().startsWith("_")) {
                                                dirs.add(entry);
                                        }



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to