You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@subversion.apache.org by Branko Čibej <br...@xbc.nu> on 2006/01/20 00:19:50 UTC

Re: svn commit: r18170 - trunk/subversion/libsvn_fs_base/bdb

maxb@tigris.org wrote:
> Author: maxb
> Date: Thu Jan 19 11:28:07 2006
> New Revision: 18170
>
> Modified:
>    trunk/subversion/libsvn_fs_base/bdb/env.c
>
> Log:
> In the BDB environment cache, implement an enumerated type using small integers
> instead of self-referential pointers, thus allowing the use of apr_atomic_cas()
> instead of apr_atomic_casptr(), thus avoiding breaking GCC's strict-aliasing
> rules, and provoking warnings.  Incidentally, it also avoids confusing humans,
> which the use of self-referential pointers merely as unique tokens is prone to
> do :-).
>   
I'm glad you agree that the spinlock idea works. :)

-- Brane


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org