[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1572363 - in /subversion/trunk/subversion: include/ libsvn_fs/ libsvn_fs_base/ libsvn_fs_fs/ libsvn_fs_x/ libsvn_repos/

From: Ben Reser <ben_at_reser.org>
Date: Fri, 28 Feb 2014 13:14:52 -0800

On 2/28/14, 1:07 PM, Stefan Fuhrmann wrote:
> Things are much worse, in fact. The old implementation
> would consider all property lists to be equal (FSFS only)
> because the representation_t structs are empty except
> for the txn_id. Hence, revision, offset and unifier would
> always match between any modified prop lists.

Ouch.

> r1573071 documents the problem in the API. Should we
> write an API errata?

Yes I think so. I don't expect anyone to have expected that old behavior.

> This is incorrect. NULL prop list reps are equal to
> empty prop lists.

Ahh that makes sense. Part of the reason I didn't commit it is what I expected
to find in the code so widely divergent from the way the code was that I wasn't
sure about what I'd done.

> r1573071 fixes the a!=a problem for in-txn items but that's
> the best we can do here without a content comparison.

Okay thanks!
Received on 2014-02-28 22:15:24 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.