From: "Parth Gala via GitGitGadget" <gitgitgadget@gmail.com>
To: git@vger.kernel.org
Cc: Parth Gala <parthpgala@gmail.com>
Subject: [PATCH 0/5] object.c: localize global the_repository variable into r
Date: Wed, 12 Feb 2020 19:19:06 +0000 [thread overview]
Message-ID: <pull.545.git.1581535151.gitgitgadget@gmail.com> (raw)
I have created this commit in response to
https://github.com/gitgitgadget/git/issues/379#issue-503866117. All the
functions in object.c which relied on 'the_repository' have been modified.
The functions calling these functions in object.c consisted calls to other
functions using 'the_repository' as well, and although I intended to use 'r'
at all those instances, I thought it would make more sense when we would
deal with their callee functions in another similar patch. What do you think
?
Signed-off-by: Parth Gala parthpgala@gmail.com [parthpgala@gmail.com]
Parth Gala (5):
object.c: get_max_object_index and get_indexed_object accept 'r'
parameter
object.c: lookup_unknown_object() accept 'r' as parameter
object.c: parse_object_or_die() accept 'r' as parameter
object.c: clear_object_flags() accept 'r' as parameter
object.c: clear_commit_marks_all() accept 'r' as parameter
builtin/checkout.c | 3 ++-
builtin/fsck.c | 8 +++++---
builtin/grep.c | 6 ++++--
builtin/index-pack.c | 5 +++--
builtin/log.c | 3 ++-
builtin/name-rev.c | 5 +++--
builtin/pack-objects.c | 3 ++-
builtin/prune.c | 3 ++-
bundle.c | 8 +++++---
http-push.c | 3 ++-
object.c | 32 ++++++++++++++++----------------
object.h | 12 ++++++------
pack-bitmap.c | 5 +++--
reachable.c | 6 ++++--
refs.c | 3 ++-
revision.c | 3 ++-
shallow.c | 13 ++++++++-----
t/helper/test-example-decorate.c | 7 ++++---
upload-pack.c | 19 ++++++++++++-------
walker.c | 3 ++-
20 files changed, 89 insertions(+), 61 deletions(-)
base-commit: 0ad714499976290d9a0229230cbe4efae930b8dc
Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-545%2FParthGala2k%2Flocalize-the_repository-variable-v1
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-545/ParthGala2k/localize-the_repository-variable-v1
Pull-Request: https://github.com/gitgitgadget/git/pull/545
--
gitgitgadget
next reply other threads:[~2020-02-12 19:19 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-02-12 19:19 Parth Gala via GitGitGadget [this message]
2020-02-12 19:19 ` [PATCH 1/5] object.c: get_max_object_index and get_indexed_object accept 'r' parameter Parth Gala via GitGitGadget
2020-02-12 20:22 ` Taylor Blau
2020-02-12 21:13 ` Eric Sunshine
2020-02-13 5:23 ` parth gala
2020-02-12 19:19 ` [PATCH 2/5] object.c: lookup_unknown_object() accept 'r' as parameter Parth Gala via GitGitGadget
2020-02-12 20:25 ` Taylor Blau
2020-02-12 21:11 ` Junio C Hamano
2020-02-13 18:00 ` Taylor Blau
2020-02-13 18:10 ` Junio C Hamano
2020-02-13 18:52 ` Jeff King
2020-02-15 0:00 ` Taylor Blau
2020-02-12 19:19 ` [PATCH 3/5] object.c: parse_object_or_die() " Parth Gala via GitGitGadget
2020-02-12 19:19 ` [PATCH 4/5] object.c: clear_object_flags() " Parth Gala via GitGitGadget
2020-02-12 19:19 ` [PATCH 5/5] object.c: clear_commit_marks_all() " Parth Gala via GitGitGadget
2020-02-12 20:18 ` [PATCH 0/5] object.c: localize global the_repository variable into r Taylor Blau
2020-02-13 5:14 ` parth gala
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
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=pull.545.git.1581535151.gitgitgadget@gmail.com \
--to=gitgitgadget@gmail.com \
--cc=git@vger.kernel.org \
--cc=parthpgala@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.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.