Mark Grosberg <mark@nolab.conman.org> writes:
> I noticed some changes to the trail functions. In particular, in
> begin_trail the trail->db_txn pointer is not explicitly set to NULL if no
> transaction is to be used. In particular this can cause abort_trail() or
> commit_trail() to get confused later on for certain platforms.
>
> In particular, even though the trail structure is allocated with
> pcalloc(), ANSI states that all-zero-bits does not equal a NULL pointer.
> So, much like the undo pointer, this must be set to NULL manually.
Hmmm. We depend on this equivalence all over Subversion, not just in
the trail code. Are you experiencing an actual problem on a platform
where all zero bits is not the same as NULL?
-Karl
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Sep 19 23:02:23 2003