Merge branch 'sv/typofix-apply-error-message'
authorJunio C Hamano <gitster@pobox.com>
Fri, 12 Dec 2014 22:31:35 +0000 (14:31 -0800)
committerJunio C Hamano <gitster@pobox.com>
Fri, 12 Dec 2014 22:31:35 +0000 (14:31 -0800)
* sv/typofix-apply-error-message:
apply: fix typo in an error message

1  2 
builtin/apply.c
diff --combined builtin/apply.c
index 6696ea4c3ffd4f7af04f3f56609a78a74a0a0b55,c201f0e6b0eb5253d3c76cf59bb7802225942edb..28d24f8e34c041c5afff4c659efbfeed4d5dc8c2
@@@ -7,7 -7,6 +7,7 @@@
   *
   */
  #include "cache.h"
 +#include "lockfile.h"
  #include "cache-tree.h"
  #include "quote.h"
  #include "blob.h"
@@@ -301,13 -300,11 +301,13 @@@ static int fuzzy_matchlines(const char 
        while ((*last2 == '\r') || (*last2 == '\n'))
                last2--;
  
 -      /* skip leading whitespace */
 -      while (isspace(*s1) && (s1 <= last1))
 -              s1++;
 -      while (isspace(*s2) && (s2 <= last2))
 -              s2++;
 +      /* skip leading whitespaces, if both begin with whitespace */
 +      if (s1 <= last1 && s2 <= last2 && isspace(*s1) && isspace(*s2)) {
 +              while (isspace(*s1) && (s1 <= last1))
 +                      s1++;
 +              while (isspace(*s2) && (s2 <= last2))
 +                      s2++;
 +      }
        /* early return if both lines are empty */
        if ((s1 > last1) && (s2 > last2))
                return 1;
@@@ -436,7 -433,7 +436,7 @@@ static unsigned long linelen(const cha
  
  static int is_dev_null(const char *str)
  {
 -      return !memcmp("/dev/null", str, 9) && isspace(str[9]);
 +      return skip_prefix(str, "/dev/null", &str) && isspace(*str);
  }
  
  #define TERM_SPACE    1
@@@ -1076,7 -1073,7 +1076,7 @@@ static int gitdiff_index(const char *li
  
        line = ptr + 2;
        ptr = strchr(line, ' ');
 -      eol = strchr(line, '\n');
 +      eol = strchrnul(line, '\n');
  
        if (!ptr || eol < ptr)
                ptr = eol;
@@@ -1282,7 -1279,9 +1282,7 @@@ static int parse_git_header(const char 
         */
        patch->def_name = git_header_name(line, len);
        if (patch->def_name && root) {
 -              char *s = xmalloc(root_len + strlen(patch->def_name) + 1);
 -              strcpy(s, root);
 -              strcpy(s + root_len, patch->def_name);
 +              char *s = xstrfmt("%s%s", root, patch->def_name);
                free(patch->def_name);
                patch->def_name = s;
        }
@@@ -1410,10 -1409,10 +1410,10 @@@ static void recount_diff(const char *li
                case '\\':
                        continue;
                case '@':
 -                      ret = size < 3 || prefixcmp(line, "@@ ");
 +                      ret = size < 3 || !starts_with(line, "@@ ");
                        break;
                case 'd':
 -                      ret = size < 5 || prefixcmp(line, "diff ");
 +                      ret = size < 5 || !starts_with(line, "diff ");
                        break;
                default:
                        ret = -1;
@@@ -1799,11 -1798,11 +1799,11 @@@ static struct fragment *parse_binary_hu
  
        *status_p = 0;
  
 -      if (!prefixcmp(buffer, "delta ")) {
 +      if (starts_with(buffer, "delta ")) {
                patch_method = BINARY_DELTA_DEFLATED;
                origlen = strtoul(buffer + 6, NULL, 10);
        }
 -      else if (!prefixcmp(buffer, "literal ")) {
 +      else if (starts_with(buffer, "literal ")) {
                patch_method = BINARY_LITERAL_DEFLATED;
                origlen = strtoul(buffer + 8, NULL, 10);
        }
@@@ -1921,66 -1920,6 +1921,66 @@@ static int parse_binary(char *buffer, u
        return used;
  }
  
 +static void prefix_one(char **name)
 +{
 +      char *old_name = *name;
 +      if (!old_name)
 +              return;
 +      *name = xstrdup(prefix_filename(prefix, prefix_length, *name));
 +      free(old_name);
 +}
 +
 +static void prefix_patch(struct patch *p)
 +{
 +      if (!prefix || p->is_toplevel_relative)
 +              return;
 +      prefix_one(&p->new_name);
 +      prefix_one(&p->old_name);
 +}
 +
 +/*
 + * include/exclude
 + */
 +
 +static struct string_list limit_by_name;
 +static int has_include;
 +static void add_name_limit(const char *name, int exclude)
 +{
 +      struct string_list_item *it;
 +
 +      it = string_list_append(&limit_by_name, name);
 +      it->util = exclude ? NULL : (void *) 1;
 +}
 +
 +static int use_patch(struct patch *p)
 +{
 +      const char *pathname = p->new_name ? p->new_name : p->old_name;
 +      int i;
 +
 +      /* Paths outside are not touched regardless of "--include" */
 +      if (0 < prefix_length) {
 +              int pathlen = strlen(pathname);
 +              if (pathlen <= prefix_length ||
 +                  memcmp(prefix, pathname, prefix_length))
 +                      return 0;
 +      }
 +
 +      /* See if it matches any of exclude/include rule */
 +      for (i = 0; i < limit_by_name.nr; i++) {
 +              struct string_list_item *it = &limit_by_name.items[i];
 +              if (!wildmatch(it->string, pathname, 0, NULL))
 +                      return (it->util != NULL);
 +      }
 +
 +      /*
 +       * If we had any include, a path that does not match any rule is
 +       * not used.  Otherwise, we saw bunch of exclude rules (or none)
 +       * and such a path is used.
 +       */
 +      return !has_include;
 +}
 +
 +
  /*
   * Read the patch text in "buffer" that extends for "size" bytes; stop
   * reading after seeing a single patch (i.e. changes to a single file).
@@@ -1996,20 -1935,21 +1996,20 @@@ static int parse_chunk(char *buffer, un
        if (offset < 0)
                return offset;
  
 -      patch->ws_rule = whitespace_rule(patch->new_name
 -                                       ? patch->new_name
 -                                       : patch->old_name);
 +      prefix_patch(patch);
 +
 +      if (!use_patch(patch))
 +              patch->ws_rule = 0;
 +      else
 +              patch->ws_rule = whitespace_rule(patch->new_name
 +                                               ? patch->new_name
 +                                               : patch->old_name);
  
        patchsize = parse_single_patch(buffer + offset + hdrsize,
                                       size - offset - hdrsize, patch);
  
        if (!patchsize) {
 -              static const char *binhdr[] = {
 -                      "Binary files ",
 -                      "Files ",
 -                      NULL,
 -              };
                static const char git_binary[] = "GIT binary patch\n";
 -              int i;
                int hd = hdrsize + offset;
                unsigned long llen = linelen(buffer + hd, size - hd);
  
                                patchsize = 0;
                }
                else if (!memcmp(" differ\n", buffer + hd + llen - 8, 8)) {
 +                      static const char *binhdr[] = {
 +                              "Binary files ",
 +                              "Files ",
 +                              NULL,
 +                      };
 +                      int i;
                        for (i = 0; binhdr[i]; i++) {
                                int len = strlen(binhdr[i]);
                                if (len < size - hd &&
@@@ -2627,7 -2561,7 +2627,7 @@@ static void update_image(struct image *
                 * NOTE: this knows that we never call remove_first_line()
                 * on anything other than pre/post image.
                 */
 -              img->line = xrealloc(img->line, nr * sizeof(*img->line));
 +              REALLOC_ARRAY(img->line, nr);
                img->line_allocated = img->line;
        }
        if (preimage_limit != postimage->nr)
@@@ -2933,7 -2867,9 +2933,7 @@@ static int apply_binary_fragment(struc
        case BINARY_LITERAL_DEFLATED:
                clear_image(img);
                img->len = fragment->size;
 -              img->buf = xmalloc(img->len+1);
 -              memcpy(img->buf, fragment->patch, img->len);
 -              img->buf[img->len] = '\0';
 +              img->buf = xmemdupz(fragment->patch, img->len);
                return 0;
        }
        return -1;
@@@ -3148,15 -3084,13 +3148,15 @@@ static void prepare_fn_table(struct pat
        }
  }
  
 -static int checkout_target(struct cache_entry *ce, struct stat *st)
 +static int checkout_target(struct index_state *istate,
 +                         struct cache_entry *ce, struct stat *st)
  {
        struct checkout costate;
  
        memset(&costate, 0, sizeof(costate));
        costate.base_dir = "";
        costate.refresh_cache = 1;
 +      costate.istate = istate;
        if (checkout_entry(ce, &costate, NULL) || lstat(ce->name, st))
                return error(_("cannot checkout %s"), ce->name);
        return 0;
@@@ -3323,7 -3257,7 +3323,7 @@@ static int load_current(struct image *i
        if (lstat(name, &st)) {
                if (errno != ENOENT)
                        return error(_("%s: %s"), name, strerror(errno));
 -              if (checkout_target(ce, &st))
 +              if (checkout_target(&the_index, ce, &st))
                        return -1;
        }
        if (verify_index_match(ce, &st))
@@@ -3477,7 -3411,7 +3477,7 @@@ static int check_preimage(struct patch 
                }
                *ce = active_cache[pos];
                if (stat_ret < 0) {
 -                      if (checkout_target(*ce, st))
 +                      if (checkout_target(&the_index, *ce, st))
                                return -1;
                }
                if (!cached && verify_index_match(*ce, st))
@@@ -3693,12 -3627,12 +3693,12 @@@ static int preimage_sha1_in_gitlink_pat
            hunk->oldpos == 1 && hunk->oldlines == 1 &&
            /* does preimage begin with the heading? */
            (preimage = memchr(hunk->patch, '\n', hunk->size)) != NULL &&
 -          !prefixcmp(++preimage, heading) &&
 +          starts_with(++preimage, heading) &&
            /* does it record full SHA-1? */
            !get_sha1_hex(preimage + sizeof(heading) - 1, sha1) &&
            preimage[sizeof(heading) + 40 - 1] == '\n' &&
            /* does the abbreviated name on the index line agree with it? */
 -          !prefixcmp(preimage + sizeof(heading) - 1, p->old_sha1_prefix))
 +          starts_with(preimage + sizeof(heading) - 1, p->old_sha1_prefix))
                return 0; /* it all looks fine */
  
        /* we may have full object name on the index line */
@@@ -3710,7 -3644,7 +3710,7 @@@ static void build_fake_ancestor(struct 
  {
        struct patch *patch;
        struct index_state result = { NULL };
 -      int fd;
 +      static struct lock_file lock;
  
        /* Once we start supporting the reverse patch, it may be
         * worth showing the new sha1 prefix, but until then...
                        if (!preimage_sha1_in_gitlink_patch(patch, sha1))
                                ; /* ok, the textual part looks sane */
                        else
-                               die("sha1 information is lacking or useless for submoule %s",
+                               die("sha1 information is lacking or useless for submodule %s",
                                    name);
                } else if (!get_sha1_blob(patch->old_sha1_prefix, sha1)) {
                        ; /* ok */
                        die ("Could not add %s to temporary index", name);
        }
  
 -      fd = open(filename, O_WRONLY | O_CREAT, 0666);
 -      if (fd < 0 || write_index(&result, fd) || close(fd))
 +      hold_lock_file_for_update(&lock, filename, LOCK_DIE_ON_ERROR);
 +      if (write_locked_index(&result, &lock, COMMIT_LOCK))
                die ("Could not write temporary index to %s", filename);
  
        discard_index(&result);
@@@ -3911,10 -3845,9 +3911,10 @@@ static void add_index_file(const char *
        ce->ce_flags = create_ce_flags(0);
        ce->ce_namelen = namelen;
        if (S_ISGITLINK(mode)) {
 -              const char *s = buf;
 +              const char *s;
  
 -              if (get_sha1_hex(s + strlen("Subproject commit "), ce->sha1))
 +              if (!skip_prefix(buf, "Subproject commit ", &s) ||
 +                  get_sha1_hex(s, ce->sha1))
                        die(_("corrupt patch for submodule %s"), path);
        } else {
                if (!cached) {
@@@ -4128,7 -4061,7 +4128,7 @@@ static int write_out_one_reject(struct 
                return error(_("cannot open %s: %s"), namebuf, strerror(errno));
  
        /* Normal git tools never deal with .rej, so do not pretend
 -       * this is a git patch by saying --git nor give extended
 +       * this is a git patch by saying --git or giving extended
         * headers.  While at it, maybe please "kompare" that wants
         * the trailing TAB and some garbage at the end of line ;-).
         */
@@@ -4193,6 -4126,64 +4193,6 @@@ static int write_out_results(struct pat
  
  static struct lock_file lock_file;
  
 -static struct string_list limit_by_name;
 -static int has_include;
 -static void add_name_limit(const char *name, int exclude)
 -{
 -      struct string_list_item *it;
 -
 -      it = string_list_append(&limit_by_name, name);
 -      it->util = exclude ? NULL : (void *) 1;
 -}
 -
 -static int use_patch(struct patch *p)
 -{
 -      const char *pathname = p->new_name ? p->new_name : p->old_name;
 -      int i;
 -
 -      /* Paths outside are not touched regardless of "--include" */
 -      if (0 < prefix_length) {
 -              int pathlen = strlen(pathname);
 -              if (pathlen <= prefix_length ||
 -                  memcmp(prefix, pathname, prefix_length))
 -                      return 0;
 -      }
 -
 -      /* See if it matches any of exclude/include rule */
 -      for (i = 0; i < limit_by_name.nr; i++) {
 -              struct string_list_item *it = &limit_by_name.items[i];
 -              if (!fnmatch(it->string, pathname, 0))
 -                      return (it->util != NULL);
 -      }
 -
 -      /*
 -       * If we had any include, a path that does not match any rule is
 -       * not used.  Otherwise, we saw bunch of exclude rules (or none)
 -       * and such a path is used.
 -       */
 -      return !has_include;
 -}
 -
 -
 -static void prefix_one(char **name)
 -{
 -      char *old_name = *name;
 -      if (!old_name)
 -              return;
 -      *name = xstrdup(prefix_filename(prefix, prefix_length, *name));
 -      free(old_name);
 -}
 -
 -static void prefix_patches(struct patch *p)
 -{
 -      if (!prefix || p->is_toplevel_relative)
 -              return;
 -      for ( ; p; p = p->next) {
 -              prefix_one(&p->new_name);
 -              prefix_one(&p->old_name);
 -      }
 -}
 -
  #define INACCURATE_EOF        (1<<0)
  #define RECOUNT               (1<<1)
  
@@@ -4218,6 -4209,8 +4218,6 @@@ static int apply_patch(int fd, const ch
                        break;
                if (apply_in_reverse)
                        reverse_patches(patch);
 -              if (prefix)
 -                      prefix_patches(patch);
                if (use_patch(patch)) {
                        patch_stats(patch);
                        *listp = patch;
        return 0;
  }
  
 -static int git_apply_config(const char *var, const char *value, void *cb)
 +static void git_apply_config(void)
  {
 -      if (!strcmp(var, "apply.whitespace"))
 -              return git_config_string(&apply_default_whitespace, var, value);
 -      else if (!strcmp(var, "apply.ignorewhitespace"))
 -              return git_config_string(&apply_default_ignorewhitespace, var, value);
 -      return git_default_config(var, value, cb);
 +      git_config_get_string_const("apply.whitespace", &apply_default_whitespace);
 +      git_config_get_string_const("apply.ignorewhitespace", &apply_default_ignorewhitespace);
 +      git_config(git_default_config, NULL);
  }
  
  static int option_parse_exclude(const struct option *opt,
@@@ -4427,7 -4422,7 +4427,7 @@@ int cmd_apply(int argc, const char **ar
  
        prefix = prefix_;
        prefix_length = prefix ? strlen(prefix) : 0;
 -      git_config(git_apply_config, NULL);
 +      git_apply_config();
        if (apply_default_whitespace)
                parse_whitespace_option(apply_default_whitespace);
        if (apply_default_ignorewhitespace)
        }
  
        if (update_index) {
 -              if (write_cache(newfd, active_cache, active_nr) ||
 -                  commit_locked_index(&lock_file))
 +              if (write_locked_index(&the_index, &lock_file, COMMIT_LOCK))
                        die(_("Unable to write new index file"));
        }