[PATCH] NullPointerException in PageViewport.java (redesign)

[Rainer Garus <ra...@arcor.de>]

If a page contains several forward links with the same id, then the method resolve of PageViewport is called for each forward link. But in the first call of resolve all forward links for the id are resolved in the for loop, so the id is removed from the unresolved hashmap of the page and the unresolved hashmap is set to null. In the second call of the method resolve there is then a NullPointerException.

Rainer Garus

Re: [PATCH] NullPointerException in PageViewport.java (redesign)

[Keiron Liddle <ke...@aftexsw.com>]

Hi Rainer,

Thanks for the patch and spotting the bug.

I will also change the area tree storage of the unresolved pages from
arraylist to a hash set so that there won't be any duplicates. This
should mean that it will only resolve an id reference once for a page.

The patch will also ensure that an error doesn't occur.

Thanks,
Keiron.

On Fri, 2002-09-13 at 23:57, Rainer Garus wrote:
> If a page contains several forward links with the same id, then the method resolve of PageViewport is called for each forward link. But in the first call of resolve all forward links for the id are resolved in the for loop, so the id is removed from the unresolved hashmap of the page and the unresolved hashmap is set to null. In the second call of the method resolve there is then a NullPointerException.
> 
> Rainer Garus
> ----
> 



---------------------------------------------------------------------
To unsubscribe, e-mail: fop-dev-unsubscribe@xml.apache.org
For additional commands, email: fop-dev-help@xml.apache.org