You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by st...@apache.org on 2014/08/03 23:35:49 UTC
svn commit: r1615455 -
/subversion/branches/authzperf/subversion/libsvn_repos/authz.c
Author: stefan2
Date: Sun Aug 3 21:35:49 2014
New Revision: 1615455
URL: http://svn.apache.org/r1615455
Log:
On the authzperf branch:
* subversion/libsvn_repos/authz.c
(lookup): Fix a typo and get rid of a redundant shortcut test.
Modified:
subversion/branches/authzperf/subversion/libsvn_repos/authz.c
Modified: subversion/branches/authzperf/subversion/libsvn_repos/authz.c
URL: http://svn.apache.org/viewvc/subversion/branches/authzperf/subversion/libsvn_repos/authz.c?rev=1615455&r1=1615454&r2=1615455&view=diff
==============================================================================
--- subversion/branches/authzperf/subversion/libsvn_repos/authz.c (original)
+++ subversion/branches/authzperf/subversion/libsvn_repos/authz.c Sun Aug 3 21:35:49 2014
@@ -1275,15 +1275,10 @@ lookup(lookup_state_t *state,
if ((state->rights.max_rights & required) != required)
return FALSE;
- /* Shortcut 2: We will fine enough rights everywhere in this sub-tree. */
+ /* Shortcut 2: We will find enough rights everywhere in this sub-tree. */
if ((state->rights.min_rights & required) == required)
return TRUE;
- /* Shortcut 3: The rights are the same everywhere in this sub-tree . */
- if ( (state->rights.min_rights & required)
- == (state->rights.max_rights & required))
- return (state->rights.min_rights & required) == required;
-
/* Extract the next segment. */
path = next_segment(segment, path);