Re: svn commit: r992905 - /subversion/branches/performance/subversion/libsvn_subr/svn_file_handle_cache.c

["Hyrum K. Wright" <hy...@mail.utexas.edu>]

Should you be using SVN_ERR_ASSERT() or one of its variants, here?

-Hyrum

On Sun, Sep 5, 2010 at 5:30 PM,  <st...@apache.org> wrote:
> Author: stefan2
> Date: Sun Sep  5 22:30:09 2010
> New Revision: 992905
>
> URL: http://svn.apache.org/viewvc?rev=992905&view=rev
> Log:
> Format fix:
> Remove spaces between function name and the opening parentheses
>
> * subversion/libsvn_subr/svn_file_handle_cache.c
>  (find_first, open_entry): fix calls to assert()
>
> Modified:
>    subversion/branches/performance/subversion/libsvn_subr/svn_file_handle_cache.c
>
> Modified: subversion/branches/performance/subversion/libsvn_subr/svn_file_handle_cache.c
> URL: http://svn.apache.org/viewvc/subversion/branches/performance/subversion/libsvn_subr/svn_file_handle_cache.c?rev=992905&r1=992904&r2=992905&view=diff
> ==============================================================================
> --- subversion/branches/performance/subversion/libsvn_subr/svn_file_handle_cache.c (original)
> +++ subversion/branches/performance/subversion/libsvn_subr/svn_file_handle_cache.c Sun Sep  5 22:30:09 2010
> @@ -363,7 +363,7 @@ find_first(svn_file_handle_cache_t *cach
>
>   /* the index must contain only used entries, i.e. those that actually
>    * contain an open APR file handle. */
> -  assert (!result || result->file);
> +  assert(!result || result->file);
>   return result;
>  }
>
> @@ -552,7 +552,7 @@ open_entry(svn_file_handle_cache__handle
>            apr_pool_t *pool)
>  {
>   /* any entry can be handed out to the application only once at any time */
> -  assert (! entry->open_handle);
> +  assert(!entry->open_handle);
>
>   /* the entry will no longer be idle */
>   remove_from_list(&cache->idle_entries, &entry->idle_link);
>
>
>