Hi Manik,

Ok, then the root of this seems to be in jboss cache ... The javadoc of org.jboss.cache.Node.getParent() says: "Returns the parent node. If this is the root node, this method returns null". But the implementation actually returns 'this' if it is the root node :-)

We'll fix the javadoc in infinispan-tree so it will match the implementation as you suggested.

Thanks,
Adrian

On 05/23/2012 05:22 PM, Manik Surtani wrote:

Hmm, I think the Javadocs may be incorrect.  The tree module was primarily written as a compatibility API with JBoss Cache and in JBoss Cache, rootNode.getParent() returns rootNode.

So the correct thing would be to correct the Javadocs.

On 23 May 2012, at 14:35, Adrian Nistor wrote:

Hi,

I spotted a potential problem in infinispan-tree. Noticed that the 
javadoc of org.infinispan.tree.Node.getParent() ("Returns the parent 
node. If this is the root node, this method returns <code>null</code>") 
does not actually match the implementation. It seems 
rootNode.getParent() == rootNode instead of being null.

Can't be sure what was the intention here, but I'm assuming the javadoc 
is correct. Can somebody confirm so I can file a bug?

Thanks,
Adrian
_______________________________________________
infinispan-dev mailing list
infinispan-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/infinispan-dev

--
Manik Surtani
manik@jboss.org
twitter.com/maniksurtani

Lead, Infinispan
http://www.infinispan.org




_______________________________________________
infinispan-dev mailing list
infinispan-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/infinispan-dev