Convert images to "fast" images if possible. Also fixed a bug where it

wasn't cleaning up properly if it failed to generate a bundle. Plus the
error reporting is all much better.


git-svn-id: svn+ssh://src.earth.threerings.net/narya/trunk@2477 542714f4-19e9-0310-aa3c-eee0fc999fb1

src/java/com/threerings/media/tile/bundle/tools/TileSetBundler.java

@@ -1,5 +1,5 @@
 //
-// $Id: TileSetBundler.java,v 1.14 2003/02/28 00:46:52 mdb Exp $
+// $Id: TileSetBundler.java,v 1.15 2003/04/27 06:34:43 mdb Exp $
 
 package com.threerings.media.tile.bundle.tools;
 
@@ -33,6 +33,7 @@ import com.samskivert.io.PersistenceException;
 
 import com.threerings.media.Log;
 import com.threerings.media.image.Colorization;
+import com.threerings.media.image.FastImageIO;
 import com.threerings.media.image.ImageUtil;
 import com.threerings.media.image.Mirage;
 
@@ -334,15 +335,16 @@ public class TileSetBundler
                     continue;
                 }
 
-                // let the jar file know what's coming
-                jar.putNextEntry(new JarEntry(imagePath));
-
                 // if this is an object tileset, we can't trim it!
                 if (set instanceof ObjectTileSet) {
                     // set the tileset up with an image provider; we
                     // need to do this so that we can trim it!
                     set.setImageProvider(improv);
 
+                    // we're going to trim it, so adjust the path
+                    imagePath = adjustImagePath(imagePath);
+                    jar.putNextEntry(new JarEntry(imagePath));
+
                     try {
                         // create a trimmed object tileset, which will
                         // write the trimmed tileset image to the jar
@@ -374,11 +376,25 @@ public class TileSetBundler
                     }
 
                 } else {
-                    // open the image and pipe it into the jar file
+                    // read the image file and convert it to our custom
-                    File imgfile = new File(
+                    // format in the bundle
-                        bundleDesc.getParent(), imagePath);
+                    File ifile = new File(bundleDesc.getParent(), imagePath);
-                    FileInputStream imgin = new FileInputStream(imgfile);
+                    try {
-                    StreamUtils.pipe(imgin, jar);
+                        BufferedImage image = ImageIO.read(ifile);
+                        if (FastImageIO.canWrite(image)) {
+                            imagePath = adjustImagePath(imagePath);
+                            jar.putNextEntry(new JarEntry(imagePath));
+                            set.setImagePath(imagePath);
+                            FastImageIO.write(image, jar);
+                        } else {
+                            jar.putNextEntry(new JarEntry(imagePath));
+                            FileInputStream imgin = new FileInputStream(ifile);
+                            StreamUtils.pipe(imgin, jar);
+                        }
+                    } catch (Exception e) {
+                        throw new NestableIOException(
+                            "Failure bundling image " + ifile + ": " + e, e);
+                    }
                 }
             }
 
@@ -395,14 +411,25 @@ public class TileSetBundler
 
             return true;
 
-        } catch (IOException ioe) {
+        } catch (Exception e) {
             // remove the incomplete jar file and rethrow the exception
-            fout.close();
+            jar.close();
-            target.delete();
+            if (!target.delete()) {
-            throw ioe;
+                Log.warning("Failed to close botched bundle '" + target + "'.");
+            }
+            String errmsg = "Failed to create bundle " + target + ": " + e;
+            throw new NestableIOException(errmsg, e);
         }
     }
 
+    /** Replaces the image suffix with <code>.raw</code>. */
+    protected String adjustImagePath (String imagePath)
+    {
+        int didx = imagePath.lastIndexOf(".");
+        return ((didx == -1) ? imagePath :
+                imagePath.substring(0, didx)) + ".raw";
+    }
+
     /** Used to parse our configuration. */
     public static class Mapping
     {