xmlgraphics-general mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jeremias Maerki <...@jeremias-maerki.ch>
Subject Re: svn commit: r979567 - in /xmlgraphics/commons/trunk: src/java/org/apache/xmlgraphics/image/loader/impl/PreloaderTIFF.java status.xml
Date Tue, 27 Jul 2010 12:58:12 GMT
Hi Vincent


On 27.07.2010 12:56:42 Vincent Hennebert wrote:
> Hi,
> 
> > Author: jeremias
> > Date: Tue Jul 27 07:50:40 2010
> > New Revision: 979567
> > 
> > URL: http://svn.apache.org/viewvc?rev=979567&view=rev
> > Log:
> > Fix for TIFFs which report zero as their resolution. This resulted in an exception.
> > Modified: 
> > xmlgraphics/commons/trunk/src/java/org/apache/xmlgraphics/image/loader/impl/PreloaderTIFF.java
> > ==============================================================================
> > @@ -127,7 +127,10 @@ public class PreloaderTIFF extends Abstr
> >                      xRes = fldx.getAsFloat(0);
> >                      yRes = fldy.getAsFloat(0);
> >                  }
> > -                if (unit == 2) {
> > +                if (xRes == 0 || yRes == 0) {
> > +                    //Some TIFFs may report 0 here which would lead to problems
> 
> Isn’t it a degenerate case that the user wants to be made aware of?

Not IMO. The users can always size the images to their wishes.

> Also, what if xRes == 0 but yRes != 0 or vice versa?

That's a very interesting question...


Jeremias Maerki


---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@xmlgraphics.apache.org
For additional commands, e-mail: general-help@xmlgraphics.apache.org


Mime
View raw message