bug-cvs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

patch src/recurse.c: redistribute whitespace


From: Terrence Enger
Subject: patch src/recurse.c: redistribute whitespace
Date: Wed, 22 Oct 2003 13:11:41 -0400

Greetings,

I offer this mostly cosmetic patch, in case you care for it.  I grant
permission to distribute this patch under the terms of the GNU Public
Licence.

The (IMO) faulty distribution of whitespace arose in rev 1.84 at
cvshome.org when converting definition of start_recursion() from
old-style to new-style declaration of parameters.  The result was one
very long line followed by several blank lines.

Terry.
Available for contract programming.



Index: src/recurse.c
===================================================================
RCS file: /home/cvs/cvs400/src/recurse.c,v
retrieving revision 1.1.1.6
retrieving revision 1.2
diff --context=5 -r1.1.1.6 -r1.2
*** src/recurse.c       9 Oct 2003 15:43:03 -0000       1.1.1.6
--- src/recurse.c       22 Oct 2003 16:53:42 -0000      1.2
***************
*** 64,85 ****
  
     Command line arguments (ARGC, ARGV) dictate the directories and
     files on which we operate.  In the special case of no arguments, we
     default to ".".  */
  int
! start_recursion (FILEPROC fileproc, FILESDONEPROC filesdoneproc,
DIRENTPROC direntproc, DIRLEAVEPROC dirleaveproc, void *callerdat, int
argc, char **argv, int local, int which, int aflag, int locktype, char
*update_preload, int dosrcs, char *repository_in)
!                       
!                                 
!                                  
!                               
!                     
! 
!              
!                 
!               
  
!     /* This specifies the kind of recursion.  There are several cases:
  
         1.  W_LOCAL is not set but W_REPOS or W_ATTIC is.  The current
         directory when we are called must be the repository and
         recursion proceeds according to what exists in the repository.
  
--- 64,82 ----
  
     Command line arguments (ARGC, ARGV) dictate the directories and
     files on which we operate.  In the special case of no arguments, we
     default to ".".  */
  int
! start_recursion (FILEPROC fileproc, FILESDONEPROC filesdoneproc,
!                DIRENTPROC direntproc, DIRLEAVEPROC dirleaveproc,
!                void *callerdat, int argc, char **argv, int local,
!                int which, int aflag, int locktype,
!                char *update_preload, int dosrcs, char *repository_in)
  
! 
!     /* `which` specifies the kind of recursion.  There are several
!        cases:
  
         1.  W_LOCAL is not set but W_REPOS or W_ATTIC is.  The current
         directory when we are called must be the repository and
         recursion proceeds according to what exists in the repository.
  
***************
*** 96,118 ****
         but we recurse into a directory if it is exists in the working
         directory *or* it exists in the repository.  If a directory
         does not exist in the working directory, the direntproc must
         either tell us to skip it (R_SKIP_ALL), or must create it (I
         think those are the only two cases).  */
-               
  
-               
-                  
-                          
                 
!     /* Keep track of the repository string.  This is only for the remote
mode,
!      * specifically, r* commands (rtag, rdiff, co, ...) where xgetwd() was
!      * used to locate the repository.  Things would break when xgetwd() was
!      * used with a symlinked repository because xgetwd() would return the
true
!      * path and in some cases this would cause the path to be printed as
other
!      * than the user specified in error messages and in other cases some of
!      * CVS's security assertions would fail.
       */
                          
  {
      int i, err = 0;
  #ifdef CLIENT_SUPPORT
--- 93,112 ----
         but we recurse into a directory if it is exists in the working
         directory *or* it exists in the repository.  If a directory
         does not exist in the working directory, the direntproc must
         either tell us to skip it (R_SKIP_ALL), or must create it (I
         think those are the only two cases).  */
  
                 
!     /* `repository_in` keeps track of the repository string.  This is
!      * only for the remote mode, specifically, r* commands (rtag,
!      * rdiff, co, ...) where xgetwd() was used to locate the
!      * repository.  Things would break when xgetwd() was used with a
!      * symlinked repository because xgetwd() would return the true
!      * path and in some cases this would cause the path to be printed
!      * as other than the user specified in error messages and in other
!      * cases some of CVS's security assertions would fail.
       */
                          
  {
      int i, err = 0;
  #ifdef CLIENT_SUPPORT






reply via email to

[Prev in Thread] Current Thread [Next in Thread]