changeset in /hg/icedtea: 2007-11-15 Lillian Angel <langel at red...

[Lillian Angel]

Matthew Flaschen wrote:
> Lillian Angel wrote:
>> 	        * rt/com/sun/image/codec/jpeg/JPEGImageDecoder.java: Likewise.
> 
> In theory, it should be possible to replace all this JPEG code with
> calls to javax.imageio.*
> 
> I started this by implementing createJPEGDecoder in the JPEGCodec class.
>  The stub of JPEGImageDecoder was actually incorrect, because the stub
> was an (incomplete) class while JPEGImageDecoder is an interface
> (http://java.sun.com/j2se/1.5.0/docs/guide/2d/api-jpeg/com/sun/image/codec/jpeg/JPEGImageDecoder.html).
>  So I wrote a proper JPEGImageDecoder interface, then used that
> implement a private class in JPEGCodec, ImageIOJPEGImageDecoder.
> 
> Finally, I made a ImageFormatException stub to match the interface.  The
> files I worked on are attached as a hg diff.

I tried building the plugs with your patch (make plugs) and I am 
getting these errors:

[...]
5. ERROR in rt/com/sun/image/codec/jpeg/JPEGImageDecoder.java (at line 57)
         public JPEGDecodeParam getJPEGDecodeParam();
                ^^^^^^^^^^^^^^^
JPEGDecodeParam cannot be resolved to a type
----------
6. ERROR in rt/com/sun/image/codec/jpeg/JPEGImageDecoder.java (at line 59)
         public void setJPEGDecodeParam(JPEGDecodeParam jdp);
                                        ^^^^^^^^^^^^^^^
JPEGDecodeParam cannot be resolved to a type
----------
6 problems (6 errors)make: *** [stamps/rt-class-files.stamp] Error 255


Lillian