atom feed10 messages in org.apache.jackrabbit.oak-commitsRe: svn commit: r1309458 - /jackrabbi...
FromSent OnAttachments
ste...@apache.orgApr 4, 2012 8:36 am 
Jukka ZittingApr 4, 2012 11:59 pm 
Stefan GuggisbergApr 5, 2012 12:43 am 
Michael DürigApr 5, 2012 1:30 am 
Jukka ZittingApr 5, 2012 1:37 am 
Thomas MuellerApr 5, 2012 1:48 am 
Jukka ZittingApr 5, 2012 1:51 am 
Thomas MuellerApr 5, 2012 2:51 am 
Jukka ZittingApr 5, 2012 2:59 am 
Thomas MuellerApr 5, 2012 7:03 am 
Subject:Re: svn commit: r1309458 - /jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/kernel/KernelNodeState.java
From:Jukka Zitting (jukk@gmail.com)
Date:Apr 5, 2012 1:37:42 am
List:org.apache.jackrabbit.oak-commits

Hi,

On Thu, Apr 5, 2012 at 9:43 AM, Stefan Guggisberg <stef@gmail.com> wrote:

you're right, but in this particular case i replaced a dummy getNodes() call in a try/catch block with a single nodeExists() call returning a boolean. i'd argue that the code is cleaner and probably not slower ;)

Right, I was confusing this with an earlier KernelNodeState design that repeated the getNodes() call in the constructor. Sorry for the noise.

BR,

Jukka Zitting