Re: svn commit: r16244 - in trunk: . subversion/clients/cmdline subversion/include subversion/libsvn_client subversion/libsvn_wc subversion/svnversion subversion/tests/clients/cmdline subversion/tests/clients/cmdline/svntest tools/examples

Branko Èibej <brane@xbc.nu> writes:

> Philip Martin wrote:
>>>+ const char **dir_name;
>>>+ for (dir_name = valid_dir_names; *dir_name; ++dir_name)
>>>+ if (0 == strcmp (name, *dir_name))
>>>+ {
>>>+ void *new_adm_dir_name = (void*) *dir_name;
>>>+ void *old_adm_dir_name = adm_dir_name;
>>>
>>Perhaps 'const void *' and lose the cast?
>>
> I need a cast _somewhere_ because apr_atomic_casptr takes non-const
> void* parameters (and the first one has to be volatile). I'll think
> about this.

Yes, I see. On Linux no casts are necessary because apr_atomic_casptr
is a #define to apr_atomic_cas and it already includes casts to long.

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