From a2ebbffca75efb33a81624c2d7f5644784e0992b Mon Sep 17 00:00:00 2001 From: Wayne Davison Date: Wed, 6 Dec 2006 01:26:11 +0000 Subject: [PATCH] - Make some char* pointers const. - Tweaked a couple variable names. - Changed the #ifdefs to not define any hard-link functions when hard-linking is disabled. --- hlink.c | 41 +++++++++++++++-------------------------- 1 file changed, 15 insertions(+), 26 deletions(-) diff --git a/hlink.c b/hlink.c index 2a22d270..89b9cc73 100644 --- a/hlink.c +++ b/hlink.c @@ -122,11 +122,9 @@ static void link_idev_data(void) the_file_list->hlink_pool = hlink_pool; pool_destroy(idev_pool); } -#endif void init_hard_links(void) { -#ifdef SUPPORT_HARD_LINKS int i; if (hlink_list) @@ -149,20 +147,18 @@ void init_hard_links(void) hlink_list = NULL; } else link_idev_data(); -#endif } -#ifdef SUPPORT_HARD_LINKS static int maybe_hard_link(struct file_struct *file, int ndx, - char *fname, int statret, STRUCT_STAT *st, - char *toname, STRUCT_STAT *to_st, + const char *fname, int statret, STRUCT_STAT *stp, + const char *toname, STRUCT_STAT *to_stp, int itemizing, enum logcode code) { if (statret == 0) { - if (st->st_dev == to_st->st_dev - && st->st_ino == to_st->st_ino) { + if (stp->st_dev == to_stp->st_dev + && stp->st_ino == to_stp->st_ino) { if (itemizing) { - itemize(file, ndx, statret, st, + itemize(file, ndx, statret, stp, ITEM_LOCAL_CHANGE | ITEM_XNAME_FOLLOWS, 0, ""); } @@ -177,16 +173,14 @@ static int maybe_hard_link(struct file_struct *file, int ndx, return -1; } } - return hard_link_one(file, ndx, fname, statret, st, toname, + return hard_link_one(file, ndx, fname, statret, stp, toname, 0, itemizing, code); } -#endif -int hard_link_check(struct file_struct *file, int ndx, char *fname, - int statret, STRUCT_STAT *st, int itemizing, +int hard_link_check(struct file_struct *file, int ndx, const char *fname, + int statret, STRUCT_STAT *stp, int itemizing, enum logcode code, int skip) { -#ifdef SUPPORT_HARD_LINKS int head; struct hlist *hl = F_HLIST(file); @@ -227,7 +221,7 @@ int hard_link_check(struct file_struct *file, int ndx, char *fname, || st2.st_ino != st3.st_ino) continue; statret = 1; - st = &st3; + stp = &st3; if (verbose < 2 || !stdout_format_has_i) { itemizing = 0; code = FNONE; @@ -237,12 +231,12 @@ int hard_link_check(struct file_struct *file, int ndx, char *fname, if (!unchanged_file(cmpbuf, file, &st3)) continue; statret = 1; - st = &st3; + stp = &st3; if (unchanged_attrs(file, &st3)) break; } while (basis_dir[++j] != NULL); } - maybe_hard_link(file, ndx, fname, statret, st, + maybe_hard_link(file, ndx, fname, statret, stp, toname, &st2, itemizing, code); if (remove_source_files == 1 && do_xfers) send_msg_int(MSG_SUCCESS, ndx); @@ -251,13 +245,11 @@ int hard_link_check(struct file_struct *file, int ndx, char *fname, hl->hlindex = SKIPPED_LINK; return 1; } -#endif return 0; } -#ifdef SUPPORT_HARD_LINKS -int hard_link_one(struct file_struct *file, int ndx, char *fname, - int statret, STRUCT_STAT *st, char *toname, int terse, +int hard_link_one(struct file_struct *file, int ndx, const char *fname, + int statret, STRUCT_STAT *stp, const char *toname, int terse, int itemizing, enum logcode code) { if (do_link(toname, fname)) { @@ -273,7 +265,7 @@ int hard_link_one(struct file_struct *file, int ndx, char *fname, } if (itemizing) { - itemize(file, ndx, statret, st, + itemize(file, ndx, statret, stp, ITEM_LOCAL_CHANGE | ITEM_XNAME_FOLLOWS, 0, terse ? "" : toname); } @@ -281,13 +273,10 @@ int hard_link_one(struct file_struct *file, int ndx, char *fname, rprintf(code, "%s => %s\n", fname, toname); return 0; } -#endif - void hard_link_cluster(struct file_struct *file, int master, int itemizing, enum logcode code, int dest_used) { -#ifdef SUPPORT_HARD_LINKS char hlink1[MAXPATHLEN]; char *hlink2; STRUCT_STAT st1, st2; @@ -320,5 +309,5 @@ void hard_link_cluster(struct file_struct *file, int master, int itemizing, send_msg_int(MSG_SUCCESS, ndx); hl->hlindex = FINISHED_LINK; } while (!(file->flags & FLAG_HLINK_LAST)); -#endif } +#endif -- 2.34.1