Mailing-List: contact oak-dev-help@jackrabbit.apache.org; run by ezmlm
Precedence: bulk
Reply-To: oak-dev@jackrabbit.apache.org
Received-SPF: pass (athena.apache.org: domain of jukka.zitting@gmail.com
 designates 209.85.214.172 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <51B08812.9070700@apache.org>
References: <20130606124129.2F45D23889BB@eris.apache.org>
 <51B08812.9070700@apache.org>
From: Jukka Zitting <jukka.zitting@gmail.com>
Date: Thu, 6 Jun 2013 16:18:36 +0300
Message-ID: 
 <CAOFYJNZEq8iiBxV5Z1enSQyf7976ybBxMFDmS3M65dhtoOqKgA@mail.gmail.com>
Subject: Re: svn commit: r1490258 -
 /jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/memory/MemoryNodeBuilder.java
To: Oak devs <oak-dev@jackrabbit.apache.org>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

Hi,

On Thu, Jun 6, 2013 at 4:01 PM, Michael D=FCrig <mduerig@apache.org> wrote:
> On 6.6.13 13:41, jukka@apache.org wrote:
>>       private Head head() {
>> -        return head.update();
>> +        Head newHead =3D head.update();
>> +        if (newHead !=3D head) {
>> +            head =3D newHead;
>> +        }
>> +        return newHead;
>>       }
>
> Isn't this the same as
>
> return head =3D head.update()

It is, except for the nice side-effect of being able to easily set a
breakpoint on state transitions.

BR,

Jukka Zitting