You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by ds...@apache.org on 2024/01/29 20:46:59 UTC

svn commit: r1915466 - /subversion/trunk/subversion/libsvn_wc/revert.c

Author: dsahlberg
Date: Mon Jan 29 20:46:59 2024
New Revision: 1915466

URL: http://svn.apache.org/viewvc?rev=1915466&view=rev
Log:
Follow-up to r1915215

* subversion/libsvn_wc/revert.c
  (revert_wc_data): *notify_access should be reset, just like *notify_required
                    is reset in revert_list_reset().
                    Fix a mistake in check for APR_UWRITE.


Modified:
    subversion/trunk/subversion/libsvn_wc/revert.c

Modified: subversion/trunk/subversion/libsvn_wc/revert.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_wc/revert.c?rev=1915466&r1=1915465&r2=1915466&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_wc/revert.c (original)
+++ subversion/trunk/subversion/libsvn_wc/revert.c Mon Jan 29 20:46:59 2024
@@ -512,6 +512,8 @@ revert_wc_data(svn_boolean_t *run_wq,
   svn_boolean_t special;
 #endif
 
+  *notify_noaccess = FALSE; /* notify_required is reset elsewhere */
+  
   /* Would be nice to use svn_io_dirent2_t here, but the performance
      improvement that provides doesn't work, because we need the read
      only and executable bits later on, in the most likely code path */
@@ -678,7 +680,7 @@ revert_wc_data(svn_boolean_t *run_wq,
 			   * Only checking for user's W since that is the only
 			   * one set by svn_io_set_file_read_write()
 			   * Issue #4622 */
-                          if (finfo.protection |  APR_UWRITE)
+                          if (finfo.protection & APR_UWRITE)
                             *notify_noaccess = TRUE;
                           else
                             {