<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
Hi Manik,<br>
<br>
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<code></code>".
But the implementation actually returns 'this' if it is the root
node :-)<br>
<br>
We'll fix the javadoc in infinispan-tree so it will match the
implementation as you suggested.<br>
<br>
Thanks,<br>
Adrian<br>
<br>
On 05/23/2012 05:22 PM, Manik Surtani wrote:
<blockquote
cite="mid:7034CFE5-68DA-4A1B-95B5-3195CED79983@jboss.org"
type="cite">
<pre wrap="">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:
</pre>
<blockquote type="cite">
<pre wrap="">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
<a class="moz-txt-link-abbreviated" href="mailto:infinispan-dev@lists.jboss.org">infinispan-dev@lists.jboss.org</a>
<a class="moz-txt-link-freetext" href="https://lists.jboss.org/mailman/listinfo/infinispan-dev">https://lists.jboss.org/mailman/listinfo/infinispan-dev</a>
</pre>
</blockquote>
<pre wrap="">
--
Manik Surtani
<a class="moz-txt-link-abbreviated" href="mailto:manik@jboss.org">manik@jboss.org</a>
twitter.com/maniksurtani
Lead, Infinispan
<a class="moz-txt-link-freetext" href="http://www.infinispan.org">http://www.infinispan.org</a>
_______________________________________________
infinispan-dev mailing list
<a class="moz-txt-link-abbreviated" href="mailto:infinispan-dev@lists.jboss.org">infinispan-dev@lists.jboss.org</a>
<a class="moz-txt-link-freetext" href="https://lists.jboss.org/mailman/listinfo/infinispan-dev">https://lists.jboss.org/mailman/listinfo/infinispan-dev</a>
</pre>
</blockquote>
<br>
</body>
</html>