The issue (which was fixed with a commit yesterday) was just that the
WorldImageWriter was always using the format "png" to write the image
world file, ignoring its own "extension" parameter in the metadata
writing method.

Patch in revision 28699 was:

---
modules/plugin/image/src/main/java/org/geotools/gce/image/WorldImageWriter.java 
    (revision 28698)
+++
modules/plugin/image/src/main/java/org/geotools/gce/image/WorldImageWriter.java 
    (working copy)
@@ -300,8 +300,7 @@
                // 
////////////////////////////////////////////////////////////////////
                final StringBuffer buff = new StringBuffer(baseFile);
 //              looking for another extension
-               final Set ext=WorldImageFormat.getWorldExtension(format
-                               
.getWriteParameters().parameter("format").stringValue());
+               final Set
ext=WorldImageFormat.getWorldExtension(this.extension);
                final Iterator it=ext.iterator();
                if(!it.hasNext())
                        throw new DataSourceException("Unable to parse
extension "+extension);



All tests are passing for me now...and Hudson too, I guess.

--saul




On Fri, 2008-01-11 at 12:32 +0100, Simone Giannecchini wrote:
> CIao guys,
> daniele is working on other things these hence you can send an email
> here so that I will be ble to try and fix the issue.
> 
> Simone.
> 
> On Jan 11, 2008 12:29 PM, Andrea Aime <[EMAIL PROTECTED]> wrote:
> > Martin Desruisseaux ha scritto:
> > > Hello Saul
> > >
> > > Saul Farber a écrit :
> > >> I've got a test failure in WorldImageWriterTest that I think I've
> > >> isolated.  You have a second to hop on irc:#geotools and work through it
> > >> with me?
> > >
> > > I'm not the author of WorlImageReader/Writer... Is the failure related to
> > > metadata, referencing or "core" coverage module? Otherwise, maybe Simone 
> > > or
> > > Andrea (I don't remember which one wrote WorlImageWriter) would be more 
> > > helpful.
> >
> > According to svn log WorldImageWriter has been modified a few days ago
> > by Daniele Romagnoli, which I'm cc'ing.
> > Yet, at least on trunk, the image module is building for me.
> > Can you provide more information about the test failure?
> >
> > Cheers
> > Andrea
> >
> >
> > -------------------------------------------------------------------------
> > Check out the new SourceForge.net Marketplace.
> > It's the best place to buy or sell services for
> > just about anything Open Source.
> > http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
> > _______________________________________________
> > Geotools-devel mailing list
> > [email protected]
> > https://lists.sourceforge.net/lists/listinfo/geotools-devel
> >
> 
> 
> 


-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
_______________________________________________
Geotools-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to