On Dec 14, 2007, at 12:49, Andreas L Delmelle wrote:

Slight correction:
On Dec 14, 2007, at 11:55, Vincent Hennebert wrote:

Hi Andreas,

+        while (!(ancestor instanceof Flow)
+                && ancestor instanceof Wrapper) {

Might be wrong, but isn’t that the very same as:
    while (!(ancestor instanceof Wrapper))
?
<snip />
Your version will run indefinitely, unless there is a Wrapper somewhere in the ancestry...

Your version will throw a NullPointerException:
when ancestor becomes the Root,
ancestor.getParent() will return null,
'ancestor instanceof Wrapper' will still return false, so the loop will be entered one more time,
and then ancestor.getParent() will throw a NPE.


Cheers

Andreas

Reply via email to