Added "debug_type" to the exclude_list_struct.
[rsync/rsync.git] / exclude.c
index 9fa8319..063f733 100644 (file)
--- a/exclude.c
+++ b/exclude.c
 #include "rsync.h"
 
 extern int verbose;
+extern int eol_nulls;
+extern int list_only;
+extern int recurse;
 
-struct exclude_struct **exclude_list;
-struct exclude_struct **local_exclude_list;
-struct exclude_struct **server_exclude_list;
+extern char curr_dir[];
+
+struct exclude_list_struct exclude_list = { 0, 0, "" };
+struct exclude_list_struct local_exclude_list = { 0, 0, "local-cvsignore " };
+struct exclude_list_struct server_exclude_list = { 0, 0, "server " };
 char *exclude_path_prefix = NULL;
 
 /** Build an exclude structure given a exclude pattern */
-static struct exclude_struct *make_exclude(const char *pattern, int pat_len,
-                                          int include)
+static void make_exclude(struct exclude_list_struct *listp, const char *pattern,
+                        int pat_len, int include)
 {
        struct exclude_struct *ret;
        const char *cp;
@@ -80,32 +85,35 @@ static struct exclude_struct *make_exclude(const char *pattern, int pat_len,
        for (cp = ret->pattern; (cp = strchr(cp, '/')) != NULL; cp++)
                ret->slash_cnt++;
 
-       return ret;
+       if (!listp->tail)
+               listp->head = listp->tail = ret;
+       else {
+               listp->tail->next = ret;
+               listp->tail = ret;
+       }
 }
 
 static void free_exclude(struct exclude_struct *ex)
 {
        free(ex->pattern);
-       memset(ex, 0, sizeof ex[0]);
        free(ex);
 }
 
-
-void free_exclude_list(struct exclude_struct ***listp)
+void free_exclude_list(struct exclude_list_struct *listp)
 {
-       struct exclude_struct **list = *listp;
-
-       if (verbose > 2)
-               rprintf(FINFO, "[%s] clearing exclude list\n", who_am_i());
+       struct exclude_struct *ent, *next;
 
-       if (!list)
-               return;
+       if (verbose > 2) {
+               rprintf(FINFO, "[%s] clearing %sexclude list\n",
+                       who_am_i(), listp->debug_type);
+       }
 
-       while (*list)
-               free_exclude(*list++);
+       for (ent = listp->head; ent; ent = next) {
+               next = ent->next;
+               free_exclude(ent);
+       }
 
-       free(*listp);
-       *listp = NULL;
+       listp->head = listp->tail = NULL;
 }
 
 static int check_one_exclude(char *name, struct exclude_struct *ex,
@@ -124,7 +132,6 @@ static int check_one_exclude(char *name, struct exclude_struct *ex,
        }
        else if ((ex->match_flags & MATCHFLG_ABS_PATH) && *name != '/') {
                static char full_name[MAXPATHLEN];
-               extern char curr_dir[];
                int plus = curr_dir[1] == '\0'? 1 : 0;
                pathjoin(full_name, sizeof full_name, curr_dir+plus, name);
                name = full_name;
@@ -191,32 +198,33 @@ static int check_one_exclude(char *name, struct exclude_struct *ex,
 
 static void report_exclude_result(char const *name,
                                   struct exclude_struct const *ent,
-                                  int name_is_dir)
+                                  int name_is_dir, const char *type)
 {
        /* If a trailing slash is present to match only directories,
         * then it is stripped out by make_exclude.  So as a special
         * case we add it back in here. */
 
        if (verbose >= 2) {
-               rprintf(FINFO, "[%s] %scluding %s %s because of pattern %s%s\n",
+               rprintf(FINFO, "[%s] %scluding %s %s because of %spattern %s%s\n",
                        who_am_i(), ent->include ? "in" : "ex",
-                       name_is_dir ? "directory" : "file", name, ent->pattern,
-                       ent->directory ? "/" : "");
+                       name_is_dir ? "directory" : "file", name, type,
+                       ent->pattern, ent->directory ? "/" : "");
        }
 }
 
 
 /*
- * Return true if file NAME is defined to be excluded by either
- * LOCAL_EXCLUDE_LIST or the globals EXCLUDE_LIST.
+ * Return true if file NAME is defined to be excluded by the specified
+ * exclude list.
  */
-int check_exclude(struct exclude_struct **list, char *name, int name_is_dir)
+int check_exclude(struct exclude_list_struct *listp, char *name, int name_is_dir)
 {
        struct exclude_struct *ent;
 
-       while ((ent = *list++) != NULL) {
+       for (ent = listp->head; ent; ent = ent->next) {
                if (check_one_exclude(name, ent, name_is_dir)) {
-                       report_exclude_result(name, ent, name_is_dir);
+                       report_exclude_result(name, ent, name_is_dir,
+                                             listp->debug_type);
                        return !ent->include;
                }
        }
@@ -236,13 +244,15 @@ int check_exclude(struct exclude_struct **list, char *name, int name_is_dir)
 static const char *get_exclude_tok(const char *p, int *len_ptr, int *incl_ptr,
                                   int xflags)
 {
-       const unsigned char *s = (unsigned char *)p;
+       const unsigned char *s = (const unsigned char *)p;
        int len;
 
        if (xflags & XFLG_WORD_SPLIT) {
                /* Skip over any initial whitespace. */
                while (isspace(*s))
                        s++;
+               /* Update for "!" check. */
+               p = (const char *)s;
        }
 
        /* Is this a '+' or '-' followed by a space (not whitespace)? */
@@ -262,8 +272,7 @@ static const char *get_exclude_tok(const char *p, int *len_ptr, int *incl_ptr,
        } else
                len = strlen(s);
 
-       if (*s == '!' && len == 1 && !(xflags & XFLG_NO_PREFIXES)
-           && (const char *)s == p)
+       if (*p == '!' && len == 1 && !(xflags & XFLG_NO_PREFIXES))
                *incl_ptr = -1;
 
        *len_ptr = len;
@@ -271,73 +280,45 @@ static const char *get_exclude_tok(const char *p, int *len_ptr, int *incl_ptr,
 }
 
 
-void add_exclude(struct exclude_struct ***listp, const char *pattern, int xflags)
+void add_exclude(struct exclude_list_struct *listp, const char *pattern,
+                int xflags)
 {
-       struct exclude_struct **list = *listp;
-       int pat_len, list_len = 0;
-       int incl, add_cnt = 1;
+       int pat_len, incl;
        const char *cp;
 
        if (!pattern)
                return;
 
-       if (xflags & XFLG_WORD_SPLIT) {
-               int add = 0;
-               /* Count maximum extra tokens we might encounter. */
-               for (cp = pattern; *cp; cp++) {
-                       if (isspace(*(unsigned char *)cp)) {
-                               add_cnt += add;
-                               add = 0;
-                       } else
-                               add = 1;
-               }
-       }
-
-       cp = get_exclude_tok(pattern, &pat_len, &incl, xflags);
-       if (!pat_len)
-               return;
-
-       /* Check for the special "!" token that clears the list.  Yes, we
-        * only honor it at the start of a XFLG_WORD_SPLIT string. */
-       if (incl < 0) {
-               free_exclude_list(listp);
-               if (!--add_cnt)
-                       return;
+       cp = pattern;
+       pat_len = 0;
+       while (1) {
                cp = get_exclude_tok(cp + pat_len, &pat_len, &incl, xflags);
                if (!pat_len)
-                       return;
-       }
-
-       if (list)
-               for ( ; list[list_len]; list_len++) {}
-
-       list = *listp = realloc_array(list, struct exclude_struct *,
-                                     list_len + add_cnt + 1);
-       if (!list)
-               out_of_memory("add_exclude");
-
-       while (pat_len) {
-               list[list_len++] = make_exclude(cp, pat_len, incl);
-
-               if (verbose > 2) {
-                       rprintf(FINFO, "[%s] add_exclude(%s,%s)\n",
-                               who_am_i(), cp,
-                               incl ? "include" : "exclude");
+                       break;
+               /* If we got the special "!" token, clear the list. */
+               if (incl < 0)
+                       free_exclude_list(listp);
+               else {
+                       make_exclude(listp, cp, pat_len, incl);
+
+                       if (verbose > 2) {
+                               rprintf(FINFO, "[%s] add_exclude(%.*s, %s%s)\n",
+                                       who_am_i(), pat_len, cp,
+                                       listp->debug_type,
+                                       incl ? "include" : "exclude");
+                       }
                }
-               cp = get_exclude_tok(cp + pat_len, &pat_len, &incl, xflags);
        }
-
-       list[list_len] = NULL;
 }
 
 
-void add_exclude_file(struct exclude_struct ***listp, const char *fname,
+void add_exclude_file(struct exclude_list_struct *listp, const char *fname,
                      int xflags)
 {
        FILE *fp;
        char line[MAXPATHLEN];
        char *eob = line + MAXPATHLEN - 1;
-       extern int eol_nulls;
+       int word_split = xflags & XFLG_WORD_SPLIT;
 
        if (!fname || !*fname)
                return;
@@ -366,18 +347,17 @@ void add_exclude_file(struct exclude_struct ***listp, const char *fname,
                                        continue;
                                break;
                        }
+                       if (word_split && isspace(ch))
+                               break;
                        if (eol_nulls? !ch : (ch == '\n' || ch == '\r'))
                                break;
                        if (s < eob)
                                *s++ = ch;
                }
                *s = '\0';
-               if (*line && *line != ';' && *line != '#') {
-                       /* Skip lines starting with semicolon or pound.
-                        * It probably wouldn't cause any harm to not skip
-                        * them but there's no need to save them. */
+               /* Skip lines starting with semicolon or pound. */
+               if (*line && *line != ';' && *line != '#')
                        add_exclude(listp, line, xflags);
-               }
                if (ch == EOF)
                        break;
        }
@@ -387,8 +367,7 @@ void add_exclude_file(struct exclude_struct ***listp, const char *fname,
 
 void send_exclude_list(int f)
 {
-       int i;
-       extern int list_only, recurse;
+       struct exclude_struct *ent;
 
        /* This is a complete hack - blame Rusty.
         *
@@ -397,24 +376,19 @@ void send_exclude_list(int f)
        if (list_only && !recurse)
                add_exclude(&exclude_list, "/*/*", 0);
 
-       if (!exclude_list) {
-               write_int(f, 0);
-               return;
-       }
-
-       for (i = 0; exclude_list[i]; i++) {
+       for (ent = exclude_list.head; ent; ent = ent->next) {
                unsigned int l;
                char p[MAXPATHLEN+1];
 
-               l = strlcpy(p, exclude_list[i]->pattern, sizeof p);
+               l = strlcpy(p, ent->pattern, sizeof p);
                if (l == 0 || l >= MAXPATHLEN)
                        continue;
-               if (exclude_list[i]->directory) {
+               if (ent->directory) {
                        p[l++] = '/';
                        p[l] = '\0';
                }
 
-               if (exclude_list[i]->include) {
+               if (ent->include) {
                        write_int(f, l + 2);
                        write_buf(f, "+ ", 2);
                } else if ((*p == '-' || *p == '+') && p[1] == ' ') {