Skip to content

replace_object.c: rename to use dash in file name

Stefan Beller git 3 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v3 Latest15 patches 1 replyIn review
  2. v216 patches 16 repliesSuperseded
  3. v1 RFC19 patches 10 repliesSuperseded

Changes in v3 from the cover letter

* interdiff below,
the only changes are renaming the variable
- struct ref_store *main_ref_store;
+ struct ref_store *refs;
in struct repository.
as well as dropping the file rename patch.
* improved commit messages from discussion on the single patches.

Patches in v3 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/15replace_object: use oidmapreplace_object.c+16 -600 replies
2/15replace-object: move replace_map to object storeobject-store.h replace-object.h replace_object.c+24 -100 replies
3/15object-store: move lookup_replace_object to replace-object.hbuiltin/mktag.c cache.h object.c +3 more+26 -190 replies
4/15replace-object: eliminate replace objects prepared flagobject-store.h replace_object.c+10 -80 replies
5/15replace-object: check_replace_refs is safe in multi repo environmentenvironment.c replace-object.h replace_object.c+5 -50 replies
6/15refs: add repository argument to get_main_ref_storebuiltin/pack-refs.c refs.c refs.h +2 more+44 -380 replies
7/15refs: add repository argument to for_each_replace_refbuiltin/replace.c refs.c refs.h +1 more+8 -40 replies
8/15replace-object: add repository argument to prepare_replace_objectreplace_object.c+4 -20 replies
9/15replace-object: add repository argument to do_lookup_replace_objectreplace-object.h replace_object.c+4 -30 replies
10/15replace-object: add repository argument to lookup_replace_objectbuiltin/mktag.c object.c replace-object.h +2 more+8 -70 replies
11/15refs: store the main ref store inside the repository structrefs.c refs.h refs/files-backend.c +1 more+9 -150 replies
12/15refs: allow for_each_replace_ref to handle arbitrary repositoriesrefs.c refs.h+3 -50 replies
13/15replace-object: allow prepare_replace_object to handle arbitrary repositoriesreplace_object.c+5 -70 replies
14/15replace-object: allow do_lookup_replace_object to handle arbitrary repositoriesreplace-object.h replace_object.c+6 -50 replies
15/15replace-object: allow lookup_replace_object to handle arbitrary repositoriesreplace-object.h+5 -50 replies

Reviews trailers per patch

1 of 15 patches has a Reviewed-by or Acked-by.

PatchRené ScharfeJunio C HamanoMichael Haggerty
1/15
2/15
3/15
4/15Helped-by Helped-by
5/15
6/15
7/15
8/15
9/15
10/15
11/15A-b
12/15
13/15
14/15
15/15

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 3 reviewers.

Read the v3 thread

Cover letter [PATCHv3 00/15] replace_object.c: rename to use dash in file name

v3:
* interdiff below,
the only changes are renaming the variable
- struct ref_store *main_ref_store;
+ struct ref_store *refs;
in struct repository.
as well as dropping the file rename patch.
* improved commit messages from discussion on the single patches.

v2:
This applies on top of a merge of
origin/bc/object-id and origin/sb/packfiles-in-repository, both of which are pending merge to master. It is also available at
https://github.com/stefanbeller/git/tree/object-store-3

* removed whitespaces as noted by Stolee
* incorporated Renes patch as the first patch of this series (It may go independently if this series takes too long)
* Adressed Erics concern regarding sloppy commit messages (removed #Conflict markers), typo in comment
* I did not drop the main_ from the ref store, yet, as asked by Duy.

Thanks,
Stefan

v1:
This applies on top of 464416a2eaadf84d2bfdf795007863d03b222b7c (sb/packfiles-in-repository).
It is also available at https://github.com/stefanbeller/git/tree/object-store-3

This series will bring the replacement mechanism (git replace) into the object store.

The first patches are cleaning up a bit, and patches 6-19 are converting one function at a time using the tick-tock pattern with the #define trick. See cfc62fc98c (sha1_file: add repository argument to link_alt_odb_entry, 2018-03-23) for explanation.

Thanks,
Stefan

René Scharfe (1):
replace_object: use oidmap

Stefan Beller (14):
replace-object: move replace_map to object store
object-store: move lookup_replace_object to replace-object.h
replace-object: eliminate replace objects prepared flag
replace-object: check_replace_refs is safe in multi repo environment
refs: add repository argument to get_main_ref_store
refs: add repository argument to for_each_replace_ref
replace-object: add repository argument to prepare_replace_object
replace-object: add repository argument to do_lookup_replace_object
replace-object: add repository argument to lookup_replace_object
refs: store the main ref store inside the repository struct
refs: allow for_each_replace_ref to handle arbitrary repositories
replace-object: allow prepare_replace_object to handle arbitrary
repositories
replace-object: allow do_lookup_replace_object to handle arbitrary
repositories
replace-object: allow lookup_replace_object to handle arbitrary
repositories

 builtin/mktag.c           |  3 +-
 builtin/pack-refs.c       |  3 +-
 builtin/replace.c         |  4 +-
 cache.h                   | 19 --------
 environment.c             |  2 +-
 object-store.h            |  8 ++++
 object.c                  |  3 +-
 refs.c                    | 80 ++++++++++++++++----------------
 refs.h                    |  4 +-
 refs/files-backend.c      |  4 --
 replace-object.h          | 36 +++++++++++++++
 replace_object.c          | 97 ++++++++++-----------------------------
 repository.h              |  3 ++
 revision.c                |  5 +-
 sha1_file.c               |  7 +--
 streaming.c               |  3 +-
 t/helper/test-ref-store.c |  3 +-
 17 files changed, 134 insertions(+), 150 deletions(-)
 create mode 100644 replace-object.h
$ git diff github/object-store-3
 diff --git c/Makefile w/Makefile
index 94e0bf47b1..de4b8f0c02 100644
--- c/Makefile
+++ w/Makefile
@@ -871,7 +871,7 @@ LIB_OBJS += refs/packed-backend.o
 LIB_OBJS += refs/ref-cache.o
 LIB_OBJS += ref-filter.o
 LIB_OBJS += remote.o
-LIB_OBJS += replace-object.o
+LIB_OBJS += replace_object.o
 LIB_OBJS += repository.o
 LIB_OBJS += rerere.o
 LIB_OBJS += resolve-undo.o
refs.c +4 -4
index bed5f88405..9b56fa9b81 100644
@@ -1651,11 +1651,11 @@ static struct ref_store *ref_store_init(const char *gitdir,
 
 struct ref_store *get_main_ref_store(struct repository *r)
 {
-       if (r->main_ref_store)
-               return r->main_ref_store;
+       if (r->refs)
+               return r->refs;
 
-       r->main_ref_store = ref_store_init(r->gitdir, REF_STORE_ALL_CAPS);
-       return r->main_ref_store;
+       r->refs = ref_store_init(r->gitdir, REF_STORE_ALL_CAPS);
+       return r->refs;
 }
 
 /*
refs/files-backend.c +0 -4
index bec8e30e9e..5c76a75817 100644
@@ -61,10 +61,6 @@ struct ref_lock {
        struct object_id old_oid;
 };
 
-/*
- * Future: need to be in "struct repository"
- * when doing a full libification.
- */
 struct files_ref_store {
        struct ref_store base;
        unsigned int store_flags;
replace_object.c renamed from replace-object.c +0 -0
similarity index 100%
rename from replace-object.c
rename to replace_object.c
repository.h +1 -1
index 7d0710b273..e6e00f541b 100644
@@ -27,7 +27,7 @@ struct repository {
        struct raw_object_store *objects;
 
        /* The store in which the refs are held. */
-       struct ref_store *main_ref_store;
+       struct ref_store *refs;
 
        /*
         * Path to the repository's graft file.
Signature · 1 line
2.17.0.484.g0c8726318c-goog