[ 
https://issues.apache.org/jira/browse/BATIK-906?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

simon steiner resolved BATIK-906.
---------------------------------
    Resolution: Fixed
      Assignee: simon steiner  (was: Batik Developer's Mailing list)

> Should never use Error to report runtime errors
> -----------------------------------------------
>
>                 Key: BATIK-906
>                 URL: https://issues.apache.org/jira/browse/BATIK-906
>             Project: Batik
>          Issue Type: Bug
>          Components: GVT
>    Affects Versions: 1.7
>         Environment: Operating System: Windows XP
> Platform: PC
>            Reporter: Sebb
>            Assignee: simon steiner
>
> TIFFImageEncoder uses Error to report unexpected conditions, e.g.
>         switch(dataType) {
>         case DataBuffer.TYPE_BYTE:
>             if(sampleSize[0] != 1 && sampleSize[0] == 4 &&
>                sampleSize[0] != 8) {
>                 throw new Error("TIFFImageEncoder2");
>             }
>             break;
> Using Error makes it hard to safely catch the exception, as there are some 
> Errors that should not be caught (e.g. ThreadDeath)
> The code should use a more specific Error, e.g. RuntimeException or 
> IllegalArgumentException or InvalidParameterException.
> There are over 70 classes where Error is thrown - please can these be changed 
> to something more appropriate?



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: batik-dev-unsubscr...@xmlgraphics.apache.org
For additional commands, e-mail: batik-dev-h...@xmlgraphics.apache.org

Reply via email to