From: <stefan.naewe@atlas-elektronik.com>
To: <christian.couder@gmail.com>, <git@vger.kernel.org>
Cc: <gitster@pobox.com>, <peff@peff.net>, <avarab@gmail.com>,
<karsten.blees@gmail.com>, <pclouds@gmail.com>,
<sbeller@google.com>, <sunshine@sunshineco.com>,
<ramsay@ramsayjones.plus.com>, <j6t@kdbg.org>, <l.s.r@web.de>,
<chriscool@tuxfamily.org>
Subject: Re: [PATCH v10 01/40] apply: make some names more specific
Date: Tue, 9 Aug 2016 14:51:20 +0000 [thread overview]
Message-ID: <50063bbb-aa5f-cf75-310e-f8305ef30d2d@atlas-elektronik.com> (raw)
In-Reply-To: <20160808210337.5038-2-chriscool@tuxfamily.org>
Am 08.08.2016 um 23:02 schrieb Christian Couder:
> To prepare for some structs and constants being moved from
> builtin/apply.c to apply.h, we should give them some more
> specific names to avoid possible name collisions in th global
s/th/the/
> namespace.
>
> Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
> ---
> builtin/apply.c | 20 ++++++++++----------
> 1 file changed, 10 insertions(+), 10 deletions(-)
>
> diff --git a/builtin/apply.c b/builtin/apply.c
> index 1a488f9..ab8f0bd 100644
> --- a/builtin/apply.c
> +++ b/builtin/apply.c
> @@ -21,7 +21,7 @@
> #include "ll-merge.h"
> #include "rerere.h"
>
> -enum ws_error_action {
> +enum apply_ws_error_action {
> nowarn_ws_error,
> warn_on_ws_error,
> die_on_ws_error,
> @@ -29,7 +29,7 @@ enum ws_error_action {
> };
>
>
> -enum ws_ignore {
> +enum apply_ws_ignore {
> ignore_ws_none,
> ignore_ws_change
> };
> @@ -45,8 +45,8 @@ enum ws_ignore {
> * See also "struct string_list symlink_changes" in "struct
> * apply_state".
> */
> -#define SYMLINK_GOES_AWAY 01
> -#define SYMLINK_IN_RESULT 02
> +#define APPLY_SYMLINK_GOES_AWAY 01
> +#define APPLY_SYMLINK_IN_RESULT 02
>
> struct apply_state {
> const char *prefix;
> @@ -110,8 +110,8 @@ struct apply_state {
> struct string_list fn_table;
>
> /* These control whitespace errors */
> - enum ws_error_action ws_error_action;
> - enum ws_ignore ws_ignore_action;
> + enum apply_ws_error_action ws_error_action;
> + enum apply_ws_ignore ws_ignore_action;
> const char *whitespace_option;
> int whitespace_error;
> int squelch_whitespace_errors;
> @@ -3750,11 +3750,11 @@ static void prepare_symlink_changes(struct apply_state *state, struct patch *pat
> if ((patch->old_name && S_ISLNK(patch->old_mode)) &&
> (patch->is_rename || patch->is_delete))
> /* the symlink at patch->old_name is removed */
> - register_symlink_changes(state, patch->old_name, SYMLINK_GOES_AWAY);
> + register_symlink_changes(state, patch->old_name, APPLY_SYMLINK_GOES_AWAY);
>
> if (patch->new_name && S_ISLNK(patch->new_mode))
> /* the symlink at patch->new_name is created or remains */
> - register_symlink_changes(state, patch->new_name, SYMLINK_IN_RESULT);
> + register_symlink_changes(state, patch->new_name, APPLY_SYMLINK_IN_RESULT);
> }
> }
>
> @@ -3769,9 +3769,9 @@ static int path_is_beyond_symlink_1(struct apply_state *state, struct strbuf *na
> break;
> name->buf[name->len] = '\0';
> change = check_symlink_changes(state, name->buf);
> - if (change & SYMLINK_IN_RESULT)
> + if (change & APPLY_SYMLINK_IN_RESULT)
> return 1;
> - if (change & SYMLINK_GOES_AWAY)
> + if (change & APPLY_SYMLINK_GOES_AWAY)
> /*
> * This cannot be "return 0", because we may
> * see a new one created at a higher level.
>
Stefan
--
----------------------------------------------------------------
/dev/random says: Don't be so humble, you're not that great. -Golda Meir
python -c "print '73746566616e2e6e616577654061746c61732d656c656b74726f6e696b2e636f6d'.decode('hex')"
GPG Key fingerprint = 2DF5 E01B 09C3 7501 BCA9 9666 829B 49C5 9221 27AF
next prev parent reply other threads:[~2016-08-09 15:05 UTC|newest]
Thread overview: 51+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-08-08 21:02 [PATCH v10 00/40] libify apply and use lib in am, part 2 Christian Couder
2016-08-08 21:02 ` [PATCH v10 01/40] apply: make some names more specific Christian Couder
2016-08-09 14:51 ` stefan.naewe [this message]
2016-08-11 8:40 ` Christian Couder
2016-08-11 8:55 ` stefan.naewe
2016-08-08 21:02 ` [PATCH v10 02/40] apply: move 'struct apply_state' to apply.h Christian Couder
2016-08-08 21:03 ` [PATCH v10 03/40] builtin/apply: make apply_patch() return -1 or -128 instead of die()ing Christian Couder
2016-08-08 21:03 ` [PATCH v10 04/40] builtin/apply: read_patch_file() return -1 " Christian Couder
2016-08-08 21:03 ` [PATCH v10 05/40] builtin/apply: make find_header() return -128 " Christian Couder
2016-08-08 21:03 ` [PATCH v10 06/40] builtin/apply: make parse_chunk() return a negative integer on error Christian Couder
2016-08-08 21:03 ` [PATCH v10 07/40] builtin/apply: make parse_single_patch() return -1 " Christian Couder
2016-08-08 21:03 ` [PATCH v10 08/40] builtin/apply: make parse_whitespace_option() return -1 instead of die()ing Christian Couder
2016-08-08 21:03 ` [PATCH v10 09/40] builtin/apply: make parse_ignorewhitespace_option() " Christian Couder
2016-08-08 21:03 ` [PATCH v10 10/40] builtin/apply: move init_apply_state() to apply.c Christian Couder
2016-08-08 21:03 ` [PATCH v10 11/40] apply: make init_apply_state() return -1 instead of exit()ing Christian Couder
2016-08-08 21:03 ` [PATCH v10 12/40] builtin/apply: make check_apply_state() return -1 instead of die()ing Christian Couder
2016-08-08 21:03 ` [PATCH v10 13/40] builtin/apply: move check_apply_state() to apply.c Christian Couder
2016-08-08 21:03 ` [PATCH v10 14/40] builtin/apply: make apply_all_patches() return 128 or 1 on error Christian Couder
2016-08-08 21:03 ` [PATCH v10 15/40] builtin/apply: make parse_traditional_patch() return -1 " Christian Couder
2016-08-08 21:03 ` [PATCH v10 16/40] builtin/apply: make gitdiff_*() return 1 at end of header Christian Couder
2016-08-08 21:03 ` [PATCH v10 17/40] builtin/apply: make gitdiff_*() return -1 on error Christian Couder
2016-08-08 21:03 ` [PATCH v10 18/40] builtin/apply: change die_on_unsafe_path() to check_unsafe_path() Christian Couder
2016-08-08 21:03 ` [PATCH v10 19/40] builtin/apply: make build_fake_ancestor() return -1 on error Christian Couder
2016-08-08 21:03 ` [PATCH v10 20/40] builtin/apply: make remove_file() " Christian Couder
2016-08-08 21:03 ` [PATCH v10 21/40] builtin/apply: make add_conflicted_stages_file() " Christian Couder
2016-08-08 21:03 ` [PATCH v10 22/40] builtin/apply: make add_index_file() " Christian Couder
2016-08-08 21:03 ` [PATCH v10 23/40] builtin/apply: make create_file() " Christian Couder
2016-08-08 21:03 ` [PATCH v10 24/40] builtin/apply: make write_out_one_result() " Christian Couder
2016-08-08 21:03 ` [PATCH v10 25/40] builtin/apply: make write_out_results() " Christian Couder
2016-08-08 21:03 ` [PATCH v10 26/40] builtin/apply: make try_create_file() " Christian Couder
2016-08-08 21:03 ` [PATCH v10 27/40] builtin/apply: make create_one_file() " Christian Couder
2016-08-08 21:03 ` [PATCH v10 28/40] builtin/apply: rename option parsing functions Christian Couder
2016-08-09 14:55 ` stefan.naewe
2016-08-08 21:03 ` [PATCH v10 29/40] apply: rename and move opt constants to apply.h Christian Couder
2016-08-08 21:03 ` [PATCH v10 31/40] apply: make some parsing functions static again Christian Couder
2016-08-08 21:03 ` [PATCH v10 32/40] apply: use error_errno() where possible Christian Couder
2016-08-08 21:03 ` [PATCH v10 33/40] environment: add set_index_file() Christian Couder
2016-08-08 22:13 ` Junio C Hamano
2016-08-10 16:52 ` Christian Couder
2016-08-10 17:34 ` Junio C Hamano
2016-08-11 19:08 ` Christian Couder
2016-08-11 19:30 ` Junio C Hamano
2016-08-08 21:03 ` [PATCH v10 34/40] apply: make it possible to silently apply Christian Couder
2016-08-08 21:03 ` [PATCH v10 35/40] apply: don't print on stdout in verbosity_silent mode Christian Couder
2016-08-08 21:03 ` [PATCH v10 36/40] usage: add set_warn_routine() Christian Couder
2016-08-08 21:03 ` [PATCH v10 37/40] usage: add get_error_routine() and get_warn_routine() Christian Couder
2016-08-08 21:03 ` [PATCH v10 38/40] apply: change error_routine when silent Christian Couder
2016-08-08 21:03 ` [PATCH v10 39/40] apply: refactor `git apply` option parsing Christian Couder
2016-08-08 21:03 ` [PATCH v10 40/40] builtin/am: use apply api in run_apply() Christian Couder
2016-08-08 21:23 ` [PATCH v10 00/40] libify apply and use lib in am, part 2 Christian Couder
2016-08-08 22:16 ` 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=50063bbb-aa5f-cf75-310e-f8305ef30d2d@atlas-elektronik.com \
--to=stefan.naewe@atlas-elektronik.com \
--cc=avarab@gmail.com \
--cc=chriscool@tuxfamily.org \
--cc=christian.couder@gmail.com \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=j6t@kdbg.org \
--cc=karsten.blees@gmail.com \
--cc=l.s.r@web.de \
--cc=pclouds@gmail.com \
--cc=peff@peff.net \
--cc=ramsay@ramsayjones.plus.com \
--cc=sbeller@google.com \
--cc=sunshine@sunshineco.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).