From: Stefan Beller <sbeller@google.com>
To: Brandon Williams <bmwill@google.com>
Cc: "git@vger.kernel.org" <git@vger.kernel.org>,
Junio C Hamano <gitster@pobox.com>,
Duy Nguyen <pclouds@gmail.com>
Subject: Re: [PATCH v3 16/27] attr: convert git_all_attrs() to use "struct attr_check"
Date: Sat, 28 Jan 2017 15:50:23 -0800 [thread overview]
Message-ID: <CAGZ79kax8Q6bD1Xm-kGAK=yTG6TYWcR3SCOjN9BeX9rzwkyaig@mail.gmail.com> (raw)
In-Reply-To: <20170128020207.179015-17-bmwill@google.com>
On Fri, Jan 27, 2017 at 6:01 PM, Brandon Williams <bmwill@google.com> wrote:
> From: Junio C Hamano <gitster@pobox.com>
>
> This updates the other two ways the attribute check is done via an
> array of "struct attr_check_item" elements. These two niches
> appear only in "git check-attr".
>
> * The caller does not know offhand what attributes it wants to ask
> about and cannot use attr_check_initl() to prepare the
> attr_check structure.
>
> * The caller may not know what attributes it wants to ask at all,
> and instead wants to learn everything that the given path has.
>
> Such a caller can call attr_check_alloc() to allocate an empty
> attr_check, and then call attr_check_append() to add attribute names
> one by one.
>
> Signed-off-by: Junio C Hamano <gitster@pobox.com>
> Signed-off-by: Stefan Beller <sbeller@google.com>
> Signed-off-by: Brandon Williams <bmwill@google.com>
> ---
> attr.c | 168 ++++++++++++++++++++++++---------------------------
> attr.h | 9 +--
> builtin/check-attr.c | 60 +++++++++---------
> 3 files changed, 112 insertions(+), 125 deletions(-)
>
> diff --git a/attr.c b/attr.c
> index de8bf35a3..40818246f 100644
> --- a/attr.c
> +++ b/attr.c
> @@ -132,75 +132,6 @@ struct git_attr *git_attr(const char *name)
> return git_attr_internal(name, strlen(name));
> }
>
> -struct attr_check *attr_check_alloc(void)
> -{
> - return xcalloc(1, sizeof(struct attr_check));
> -}
> -
> -struct attr_check *attr_check_initl(const char *one, ...)
> -{
> - struct attr_check *check;
> - int cnt;
> - va_list params;
> - const char *param;
> -
> - va_start(params, one);
> - for (cnt = 1; (param = va_arg(params, const char *)) != NULL; cnt++)
> - ;
> - va_end(params);
> -
> - check = attr_check_alloc();
> - check->nr = cnt;
> - check->alloc = cnt;
> - check->items = xcalloc(cnt, sizeof(struct attr_check_item));
> -
> - check->items[0].attr = git_attr(one);
> - va_start(params, one);
> - for (cnt = 1; cnt < check->nr; cnt++) {
> - const struct git_attr *attr;
> - param = va_arg(params, const char *);
> - if (!param)
> - die("BUG: counted %d != ended at %d",
> - check->nr, cnt);
> - attr = git_attr(param);
> - if (!attr)
> - die("BUG: %s: not a valid attribute name", param);
> - check->items[cnt].attr = attr;
> - }
> - va_end(params);
> - return check;
> -}
This being moved down to below (being review churn) sounds like a
rebase mistake. ;)
> -
> -struct attr_check_item *attr_check_append(struct attr_check *check,
> - const struct git_attr *attr)
> -{
> - struct attr_check_item *item;
> -
> - ALLOC_GROW(check->items, check->nr + 1, check->alloc);
> - item = &check->items[check->nr++];
> - item->attr = attr;
> - return item;
> -}
> -
> -void attr_check_reset(struct attr_check *check)
> -{
> - check->nr = 0;
> -}
> -
> -void attr_check_clear(struct attr_check *check)
> -{
> - free(check->items);
> - check->items = NULL;
> - check->alloc = 0;
> - check->nr = 0;
> -}
> -
> -void attr_check_free(struct attr_check *check)
> -{
> - attr_check_clear(check);
> - free(check);
> -}
> -
> /* What does a matched pattern decide? */
> struct attr_state {
> struct git_attr *attr;
> @@ -439,6 +370,75 @@ static void free_attr_elem(struct attr_stack *e)
> free(e);
> }
>
> +struct attr_check *attr_check_alloc(void)
> +{
> + return xcalloc(1, sizeof(struct attr_check));
> +}
> +
> +struct attr_check *attr_check_initl(const char *one, ...)
> +{
> + struct attr_check *check;
> + int cnt;
> + va_list params;
> + const char *param;
> +
> + va_start(params, one);
> + for (cnt = 1; (param = va_arg(params, const char *)) != NULL; cnt++)
> + ;
> + va_end(params);
> +
> + check = attr_check_alloc();
> + check->nr = cnt;
> + check->alloc = cnt;
> + check->items = xcalloc(cnt, sizeof(struct attr_check_item));
> +
> + check->items[0].attr = git_attr(one);
> + va_start(params, one);
> + for (cnt = 1; cnt < check->nr; cnt++) {
> + const struct git_attr *attr;
> + param = va_arg(params, const char *);
> + if (!param)
> + die("BUG: counted %d != ended at %d",
> + check->nr, cnt);
> + attr = git_attr(param);
> + if (!attr)
> + die("BUG: %s: not a valid attribute name", param);
> + check->items[cnt].attr = attr;
> + }
> + va_end(params);
> + return check;
> +}
> +
> +struct attr_check_item *attr_check_append(struct attr_check *check,
> + const struct git_attr *attr)
> +{
> + struct attr_check_item *item;
> +
> + ALLOC_GROW(check->items, check->nr + 1, check->alloc);
> + item = &check->items[check->nr++];
> + item->attr = attr;
> + return item;
> +}
> +
> +void attr_check_reset(struct attr_check *check)
> +{
> + check->nr = 0;
> +}
> +
> +void attr_check_clear(struct attr_check *check)
> +{
> + free(check->items);
> + check->items = NULL;
> + check->alloc = 0;
> + check->nr = 0;
> +}
> +
> +void attr_check_free(struct attr_check *check)
> +{
> + attr_check_clear(check);
> + free(check);
> +}
> +
> static const char *builtin_attr[] = {
> "[attr]binary -diff -merge -text",
> NULL,
> @@ -906,32 +906,22 @@ int git_check_attrs(const char *path, int num, struct attr_check_item *check)
> return 0;
> }
Below is where the actual change for this patch starts?
>
> -int git_all_attrs(const char *path, int *num, struct attr_check_item **check)
> +void git_all_attrs(const char *path, struct attr_check *check)
> {
> - int i, count, j;
> + int i;
>
> - collect_some_attrs(path, 0, NULL);
> + attr_check_reset(check);
> + collect_some_attrs(path, check->nr, check->items);
>
> - /* Count the number of attributes that are set. */
> - count = 0;
> - for (i = 0; i < attr_nr; i++) {
> - const char *value = check_all_attr[i].value;
> - if (value != ATTR__UNSET && value != ATTR__UNKNOWN)
> - ++count;
> - }
> - *num = count;
> - ALLOC_ARRAY(*check, count);
> - j = 0;
> for (i = 0; i < attr_nr; i++) {
> + const char *name = check_all_attr[i].attr->name;
> const char *value = check_all_attr[i].value;
> - if (value != ATTR__UNSET && value != ATTR__UNKNOWN) {
> - (*check)[j].attr = check_all_attr[i].attr;
> - (*check)[j].value = value;
> - ++j;
> - }
> + struct attr_check_item *item;
> + if (value == ATTR__UNSET || value == ATTR__UNKNOWN)
> + continue;
> + item = attr_check_append(check, git_attr(name));
> + item->value = value;
> }
> -
> - return 0;
> }
>
> int git_check_attr(const char *path, struct attr_check *check)
> diff --git a/attr.h b/attr.h
> index e611b139a..9f2729842 100644
> --- a/attr.h
> +++ b/attr.h
> @@ -56,13 +56,10 @@ int git_check_attrs(const char *path, int, struct attr_check_item *);
> extern int git_check_attr(const char *path, struct attr_check *check);
>
> /*
> - * Retrieve all attributes that apply to the specified path. *num
> - * will be set to the number of attributes on the path; **check will
> - * be set to point at a newly-allocated array of git_attr_check
> - * objects describing the attributes and their values. *check must be
> - * free()ed by the caller.
> + * Retrieve all attributes that apply to the specified path.
> + * check holds the attributes and their values.
> */
> -int git_all_attrs(const char *path, int *num, struct attr_check_item **check);
> +extern void git_all_attrs(const char *path, struct attr_check *check);
>
> enum git_attr_direction {
> GIT_ATTR_CHECKIN,
> diff --git a/builtin/check-attr.c b/builtin/check-attr.c
> index 889264a5b..40cdff13e 100644
> --- a/builtin/check-attr.c
> +++ b/builtin/check-attr.c
> @@ -24,12 +24,13 @@ static const struct option check_attr_options[] = {
> OPT_END()
> };
>
> -static void output_attr(int cnt, struct attr_check_item *check,
> - const char *file)
> +static void output_attr(struct attr_check *check, const char *file)
> {
> int j;
> + int cnt = check->nr;
> +
> for (j = 0; j < cnt; j++) {
> - const char *value = check[j].value;
> + const char *value = check->items[j].value;
>
> if (ATTR_TRUE(value))
> value = "set";
> @@ -42,36 +43,38 @@ static void output_attr(int cnt, struct attr_check_item *check,
> printf("%s%c" /* path */
> "%s%c" /* attrname */
> "%s%c" /* attrvalue */,
> - file, 0, git_attr_name(check[j].attr), 0, value, 0);
> + file, 0,
> + git_attr_name(check->items[j].attr), 0, value, 0);
> } else {
> quote_c_style(file, NULL, stdout, 0);
> - printf(": %s: %s\n", git_attr_name(check[j].attr), value);
> + printf(": %s: %s\n",
> + git_attr_name(check->items[j].attr), value);
> }
> -
> }
> }
>
> static void check_attr(const char *prefix,
> - int cnt, struct attr_check_item *check,
> + struct attr_check *check,
> + int collect_all,
> const char *file)
> {
> char *full_path =
> prefix_path(prefix, prefix ? strlen(prefix) : 0, file);
> - if (check != NULL) {
> - if (git_check_attrs(full_path, cnt, check))
> - die("git_check_attrs died");
> - output_attr(cnt, check, file);
> +
> + if (collect_all) {
> + git_all_attrs(full_path, check);
> } else {
> - if (git_all_attrs(full_path, &cnt, &check))
> - die("git_all_attrs died");
> - output_attr(cnt, check, file);
> - free(check);
> + if (git_check_attr(full_path, check))
> + die("git_check_attr died");
> }
> + output_attr(check, file);
> +
> free(full_path);
> }
>
> static void check_attr_stdin_paths(const char *prefix,
> - int cnt, struct attr_check_item *check)
> + struct attr_check *check,
> + int collect_all)
> {
> struct strbuf buf = STRBUF_INIT;
> struct strbuf unquoted = STRBUF_INIT;
> @@ -85,7 +88,7 @@ static void check_attr_stdin_paths(const char *prefix,
> die("line is badly quoted");
> strbuf_swap(&buf, &unquoted);
> }
> - check_attr(prefix, cnt, check, buf.buf);
> + check_attr(prefix, check, collect_all, buf.buf);
> maybe_flush_or_die(stdout, "attribute to stdout");
> }
> strbuf_release(&buf);
> @@ -100,7 +103,7 @@ static NORETURN void error_with_usage(const char *msg)
>
> int cmd_check_attr(int argc, const char **argv, const char *prefix)
> {
> - struct attr_check_item *check;
> + struct attr_check *check;
> int cnt, i, doubledash, filei;
>
> if (!is_bare_repository())
> @@ -160,28 +163,25 @@ int cmd_check_attr(int argc, const char **argv, const char *prefix)
> error_with_usage("No file specified");
> }
>
> - if (all_attrs) {
> - check = NULL;
> - } else {
> - check = xcalloc(cnt, sizeof(*check));
> + check = attr_check_alloc();
> + if (!all_attrs) {
> for (i = 0; i < cnt; i++) {
> - const char *name;
> - struct git_attr *a;
> - name = argv[i];
> - a = git_attr(name);
> + struct git_attr *a = git_attr(argv[i]);
> if (!a)
> return error("%s: not a valid attribute name",
> - name);
> - check[i].attr = a;
> + argv[i]);
> + attr_check_append(check, a);
> }
> }
>
> if (stdin_paths)
> - check_attr_stdin_paths(prefix, cnt, check);
> + check_attr_stdin_paths(prefix, check, all_attrs);
> else {
> for (i = filei; i < argc; i++)
> - check_attr(prefix, cnt, check, argv[i]);
> + check_attr(prefix, check, all_attrs, argv[i]);
> maybe_flush_or_die(stdout, "attribute to stdout");
> }
> +
> + attr_check_free(check);
> return 0;
> }
> --
> 2.11.0.483.g087da7b7c-goog
>
next prev parent reply other threads:[~2017-01-28 23:59 UTC|newest]
Thread overview: 111+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-01-12 23:53 [PATCH 00/27] Revamp the attribute system; another round Brandon Williams
2017-01-12 23:53 ` [PATCH 01/27] commit.c: use strchrnul() to scan for one line Brandon Williams
2017-01-12 23:53 ` [PATCH 02/27] attr.c: " Brandon Williams
2017-01-12 23:53 ` [PATCH 03/27] attr.c: update a stale comment on "struct match_attr" Brandon Williams
2017-01-12 23:53 ` [PATCH 04/27] attr.c: explain the lack of attr-name syntax check in parse_attr() Brandon Williams
2017-01-12 23:53 ` [PATCH 05/27] attr.c: complete a sentence in a comment Brandon Williams
2017-01-12 23:53 ` [PATCH 06/27] attr.c: mark where #if DEBUG ends more clearly Brandon Williams
2017-01-12 23:53 ` [PATCH 07/27] attr.c: simplify macroexpand_one() Brandon Williams
2017-01-12 23:53 ` [PATCH 08/27] attr.c: tighten constness around "git_attr" structure Brandon Williams
2017-01-12 23:53 ` [PATCH 09/27] attr.c: plug small leak in parse_attr_line() Brandon Williams
2017-01-12 23:53 ` [PATCH 10/27] attr: support quoting pathname patterns in C style Brandon Williams
2017-01-12 23:53 ` [PATCH 11/27] attr.c: add push_stack() helper Brandon Williams
2017-01-12 23:53 ` [PATCH 12/27] Documentation: fix a typo Brandon Williams
2017-01-12 23:53 ` [PATCH 13/27] attr.c: outline the future plans by heavily commenting Brandon Williams
2017-01-12 23:53 ` [PATCH 14/27] attr: rename function and struct related to checking attributes Brandon Williams
2017-01-12 23:53 ` [PATCH 15/27] attr: (re)introduce git_check_attr() and struct attr_check Brandon Williams
2017-01-12 23:53 ` [PATCH 16/27] attr: convert git_all_attrs() to use "struct attr_check" Brandon Williams
2017-01-12 23:53 ` [PATCH 17/27] attr: convert git_check_attrs() callers to use the new API Brandon Williams
2017-01-12 23:53 ` [PATCH 18/27] attr: retire git_check_attrs() API Brandon Williams
2017-01-12 23:53 ` [PATCH 19/27] attr: pass struct attr_check to collect_some_attrs Brandon Williams
2017-01-12 23:53 ` [PATCH 20/27] attr: change validity check for attribute names to use positive logic Brandon Williams
2017-01-12 23:53 ` [PATCH 21/27] attr: use hashmap for attribute dictionary Brandon Williams
2017-01-18 20:20 ` Stefan Beller
2017-01-18 20:23 ` Brandon Williams
2017-01-12 23:53 ` [PATCH 22/27] attr: eliminate global check_all_attr array Brandon Williams
2017-01-12 23:53 ` [PATCH 23/27] attr: remove maybe-real, maybe-macro from git_attr Brandon Williams
2017-01-12 23:53 ` [PATCH 24/27] attr: tighten const correctness with git_attr and match_attr Brandon Williams
2017-01-12 23:53 ` [PATCH 25/27] attr: store attribute stacks in hashmap Brandon Williams
2017-01-13 21:20 ` Junio C Hamano
2017-01-18 20:34 ` Brandon Williams
2017-01-23 18:08 ` Brandon Williams
2017-01-18 20:39 ` Stefan Beller
2017-01-18 20:45 ` Stefan Beller
2017-01-18 20:50 ` Brandon Williams
2017-01-12 23:53 ` [PATCH 26/27] attr: push the bare repo check into read_attr() Brandon Williams
2017-01-12 23:53 ` [PATCH 27/27] attr: reformat git_attr_set_direction() function Brandon Williams
2017-01-15 23:47 ` [PATCH 00/27] Revamp the attribute system; another round Junio C Hamano
2017-01-16 8:10 ` Jeff King
2017-01-23 20:34 ` [PATCH v2 " Brandon Williams
2017-01-23 20:34 ` [PATCH v2 01/27] commit.c: use strchrnul() to scan for one line Brandon Williams
2017-01-23 20:35 ` [PATCH v2 02/27] attr.c: " Brandon Williams
2017-01-23 20:35 ` [PATCH v2 03/27] attr.c: update a stale comment on "struct match_attr" Brandon Williams
2017-01-23 20:35 ` [PATCH v2 04/27] attr.c: explain the lack of attr-name syntax check in parse_attr() Brandon Williams
2017-01-23 20:35 ` [PATCH v2 05/27] attr.c: complete a sentence in a comment Brandon Williams
2017-01-23 20:35 ` [PATCH v2 06/27] attr.c: mark where #if DEBUG ends more clearly Brandon Williams
2017-01-23 20:35 ` [PATCH v2 07/27] attr.c: simplify macroexpand_one() Brandon Williams
2017-01-23 20:35 ` [PATCH v2 08/27] attr.c: tighten constness around "git_attr" structure Brandon Williams
2017-01-23 20:35 ` [PATCH v2 09/27] attr.c: plug small leak in parse_attr_line() Brandon Williams
2017-01-23 20:35 ` [PATCH v2 10/27] attr: support quoting pathname patterns in C style Brandon Williams
2017-01-23 20:35 ` [PATCH v2 11/27] attr.c: add push_stack() helper Brandon Williams
2017-01-23 20:35 ` [PATCH v2 12/27] Documentation: fix a typo Brandon Williams
2017-01-23 20:35 ` [PATCH v2 13/27] attr.c: outline the future plans by heavily commenting Brandon Williams
2017-01-23 20:35 ` [PATCH v2 14/27] attr: rename function and struct related to checking attributes Brandon Williams
2017-01-23 20:35 ` [PATCH v2 15/27] attr: (re)introduce git_check_attr() and struct attr_check Brandon Williams
2017-01-23 20:35 ` [PATCH v2 16/27] attr: convert git_all_attrs() to use "struct attr_check" Brandon Williams
2017-01-23 20:35 ` [PATCH v2 17/27] attr: convert git_check_attrs() callers to use the new API Brandon Williams
2017-01-23 20:35 ` [PATCH v2 18/27] attr: retire git_check_attrs() API Brandon Williams
2017-01-23 20:35 ` [PATCH v2 19/27] attr: pass struct attr_check to collect_some_attrs Brandon Williams
2017-01-23 20:35 ` [PATCH v2 20/27] attr: change validity check for attribute names to use positive logic Brandon Williams
2017-01-23 20:35 ` [PATCH v2 21/27] attr: use hashmap for attribute dictionary Brandon Williams
2017-01-23 20:35 ` [PATCH v2 22/27] attr: eliminate global check_all_attr array Brandon Williams
2017-01-23 21:11 ` Junio C Hamano
2017-01-23 20:35 ` [PATCH v2 23/27] attr: remove maybe-real, maybe-macro from git_attr Brandon Williams
2017-01-23 20:35 ` [PATCH v2 24/27] attr: tighten const correctness with git_attr and match_attr Brandon Williams
2017-01-23 20:35 ` [PATCH v2 25/27] attr: store attribute stack in attr_check structure Brandon Williams
2017-01-23 21:42 ` Junio C Hamano
2017-01-23 22:06 ` Brandon Williams
2017-01-24 1:11 ` Brandon Williams
2017-01-24 2:28 ` Junio C Hamano
2017-01-25 19:57 ` Brandon Williams
2017-01-25 20:10 ` Stefan Beller
2017-01-25 20:14 ` Junio C Hamano
2017-01-25 21:54 ` Brandon Williams
2017-01-25 23:19 ` Brandon Williams
2017-01-23 20:35 ` [PATCH v2 26/27] attr: push the bare repo check into read_attr() Brandon Williams
2017-01-23 20:35 ` [PATCH v2 27/27] attr: reformat git_attr_set_direction() function Brandon Williams
2017-01-28 2:01 ` [PATCH v3 00/27] Revamp the attribute system; another round Brandon Williams
2017-01-28 2:01 ` [PATCH v3 01/27] commit.c: use strchrnul() to scan for one line Brandon Williams
2017-01-28 2:01 ` [PATCH v3 02/27] attr.c: " Brandon Williams
2017-01-28 2:01 ` [PATCH v3 03/27] attr.c: update a stale comment on "struct match_attr" Brandon Williams
2017-01-28 2:01 ` [PATCH v3 04/27] attr.c: explain the lack of attr-name syntax check in parse_attr() Brandon Williams
2017-01-28 2:01 ` [PATCH v3 05/27] attr.c: complete a sentence in a comment Brandon Williams
2017-01-28 2:01 ` [PATCH v3 06/27] attr.c: mark where #if DEBUG ends more clearly Brandon Williams
2017-01-28 2:01 ` [PATCH v3 07/27] attr.c: simplify macroexpand_one() Brandon Williams
2017-01-28 2:01 ` [PATCH v3 08/27] attr.c: tighten constness around "git_attr" structure Brandon Williams
2017-01-28 2:01 ` [PATCH v3 09/27] attr.c: plug small leak in parse_attr_line() Brandon Williams
2017-01-28 2:01 ` [PATCH v3 10/27] attr: support quoting pathname patterns in C style Brandon Williams
2017-01-28 2:01 ` [PATCH v3 11/27] attr.c: add push_stack() helper Brandon Williams
2017-01-28 2:01 ` [PATCH v3 12/27] Documentation: fix a typo Brandon Williams
2017-01-28 2:01 ` [PATCH v3 13/27] attr.c: outline the future plans by heavily commenting Brandon Williams
2017-01-28 2:01 ` [PATCH v3 14/27] attr: rename function and struct related to checking attributes Brandon Williams
2017-01-28 2:01 ` [PATCH v3 15/27] attr: (re)introduce git_check_attr() and struct attr_check Brandon Williams
2017-01-30 18:05 ` Brandon Williams
2017-01-28 2:01 ` [PATCH v3 16/27] attr: convert git_all_attrs() to use "struct attr_check" Brandon Williams
2017-01-28 23:50 ` Stefan Beller [this message]
2017-01-29 2:44 ` Brandon Williams
2017-01-30 18:06 ` Brandon Williams
2017-01-28 2:01 ` [PATCH v3 17/27] attr: convert git_check_attrs() callers to use the new API Brandon Williams
2017-01-28 2:01 ` [PATCH v3 18/27] attr: retire git_check_attrs() API Brandon Williams
2017-01-28 2:01 ` [PATCH v3 19/27] attr: pass struct attr_check to collect_some_attrs Brandon Williams
2017-01-28 2:02 ` [PATCH v3 20/27] attr: change validity check for attribute names to use positive logic Brandon Williams
2017-01-28 2:02 ` [PATCH v3 21/27] attr: use hashmap for attribute dictionary Brandon Williams
2017-01-28 2:02 ` [PATCH v3 22/27] attr: eliminate global check_all_attr array Brandon Williams
2017-01-28 2:02 ` [PATCH v3 23/27] attr: remove maybe-real, maybe-macro from git_attr Brandon Williams
2017-01-28 2:02 ` [PATCH v3 24/27] attr: tighten const correctness with git_attr and match_attr Brandon Williams
2017-01-28 2:02 ` [PATCH v3 25/27] attr: store attribute stack in attr_check structure Brandon Williams
2017-01-28 2:02 ` [PATCH v3 26/27] attr: push the bare repo check into read_attr() Brandon Williams
2017-01-28 2:02 ` [PATCH v3 27/27] attr: reformat git_attr_set_direction() function Brandon Williams
2017-02-02 19:14 ` [PATCH v3 00/27] Revamp the attribute system; another round Junio C Hamano
2017-02-09 17:18 ` Brandon Williams
2017-02-09 19:31 ` Junio C Hamano
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: http://vger.kernel.org/majordomo-info.html
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAGZ79kax8Q6bD1Xm-kGAK=yTG6TYWcR3SCOjN9BeX9rzwkyaig@mail.gmail.com' \
--to=sbeller@google.com \
--cc=bmwill@google.com \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=pclouds@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).