Re: [abc] soot doesn't build against recent polyglot

On Thu, Dec 23, 2004 at 12:30:30AM +0000, Ganesh Sittampalam wrote:
> On Wed, 22 Dec 2004, Ganesh Sittampalam wrote:
>
> > I'll make the change to Soot and test it against both Polyglot 1.3 and the
> > abc version before checking in, unless you'd rather do it.
>
> I've now written the change. I tested (a) that Soot built with Polyglot
> 1.3 can compile a trivial Java file and (b) that abc and Soot, both built
> with our snapshot of Polyglot, pass the test suite.
>
> Unfortunately, it required a rather nasty hack. Apparently it's an error
> to catch a checked exception that can't be thrown:
>
> FileSource source = new FileSource(new File(fileName));
> // This hack is to stop the catch block at the bottom causing an error
> // with versions of Polyglot where the constructor above can't throw IOException
> // It should be removed as soon as Polyglot 1.3 is no longer supported.
> if(false) throw new IOException("Bogus exception");
>
> (this goes in JavaToJimple.java near the top of the compile method)
>
> Please let me know if you're not happy with it, otherwise I'll check it in
> to Soot.

I think the hack is fine, particularly since it's clearly commented.

Ondrej

> I'm going to my parents for Christmas tomorrow evening, so I think the
> release will be next week now since I don't want to do it in a rush and
> potentially make a mess of it.
>
> Ganesh
>
>
Received on Thu Dec 23 17:48:48 2004