git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH v4 11/16] midx: make some functions non-static

From
Taylor Blau <me@ttaylorr.com>
Date
Mar 30, 2021, 15:04 UTC
Message-ID
<2ada3973202feb737af6c8cbeafae0241afdff90.1617116623.git.me@ttaylorr.com>
In-Reply-To
<cover.1617116623.git.me@ttaylorr.com>

In a subsequent commit, pack-revindex.c will become responsible for sorting a list of objects in the "MIDX pack order" (which will be defined in the following patch). To do so, it will need to be know the pack identifier and offset within that pack for each object in the MIDX.

The MIDX code already has functions for doing just that (nth_midxed_offset() and nth_midxed_pack_int_id()), but they are statically declared.

Since there is no reason that they couldn't be exposed publicly, and because they are already doing exactly what the caller in pack-revindex.c will want, expose them publicly so that they can be reused there.

Signed-off-by: Taylor Blau <me@ttaylorr.com>
---
 midx.c | 4 ++--
 midx.h | 2 ++
 2 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/midx.c b/midx.c
index 526795ff0e..2173a9b45c 100644
--- a/midx.c
+++ b/midx.c
@@ -239,7 +239,7 @@ struct object_id *nth_midxed_object_oid(struct object_id *oid,
 	return oid;
 }
 
-static off_t nth_midxed_offset(struct multi_pack_index *m, uint32_t pos)
+off_t nth_midxed_offset(struct multi_pack_index *m, uint32_t pos)
 {
 	const unsigned char *offset_data;
 	uint32_t offset32;
@@ -258,7 +258,7 @@ static off_t nth_midxed_offset(struct multi_pack_index *m, uint32_t pos)
 	return offset32;
 }
 
-static uint32_t nth_midxed_pack_int_id(struct multi_pack_index *m, uint32_t pos)
+uint32_t nth_midxed_pack_int_id(struct multi_pack_index *m, uint32_t pos)
 {
 	return get_be32(m->chunk_object_offsets +
 			(off_t)pos * MIDX_CHUNK_OFFSET_WIDTH);
diff --git a/midx.h b/midx.h
index e7fea61109..93bd68189e 100644
--- a/midx.h
+++ b/midx.h
@@ -40,6 +40,8 @@ struct multi_pack_index {
 struct multi_pack_index *load_multi_pack_index(const char *object_dir, int local);
 int prepare_midx_pack(struct repository *r, struct multi_pack_index *m, uint32_t pack_int_id);
 int bsearch_midx(const struct object_id *oid, struct multi_pack_index *m, uint32_t *result);
+off_t nth_midxed_offset(struct multi_pack_index *m, uint32_t pos);
+uint32_t nth_midxed_pack_int_id(struct multi_pack_index *m, uint32_t pos);
 struct object_id *nth_midxed_object_oid(struct object_id *oid,
 					struct multi_pack_index *m,
 					uint32_t n);
-- 
2.30.0.667.g81c0cbc6fd
Previous: Taylor BlauNext: Taylor Blau
Message 155 of 171 in “midx: implement a multi-pack reverse index”
  1. 0/9 midx: implement a multi-pack reverse indexTaylor Blau, Feb 10, 2021
  2. 1/9 t/helper/test-read-midx.c: add '--show-objects'Taylor Blau, Feb 10, 2021
  3. Derrick StoleeFeb 11, 2021
  4. Taylor BlauFeb 11, 2021
  5. 2/9 midx: allow marking a pack as preferredTaylor Blau, Feb 10, 2021
  6. SZEDER GáborFeb 11, 2021
  7. Taylor BlauFeb 15, 2021
  8. Ævar Arnfjörð BjarmasonFeb 15, 2021
  9. 0/5 commit-graph: parse_options() cleanupÆvar Arnfjörð Bjarmason, Feb 15, 2021
  10. 1/5 commit-graph: define common usage with a macroÆvar Arnfjörð Bjarmason, Feb 15, 2021
  11. Derrick StoleeFeb 16, 2021
  12. 2/5 commit-graph: remove redundant handling of -hÆvar Arnfjörð Bjarmason, Feb 15, 2021
  13. Derrick StoleeFeb 16, 2021
  14. 4/5 commit-graph: refactor dispatch loop for styleÆvar Arnfjörð Bjarmason, Feb 15, 2021
  15. Taylor BlauFeb 15, 2021
  16. Derrick StoleeFeb 16, 2021
  17. Ævar Arnfjörð BjarmasonFeb 16, 2021
  18. Derrick StoleeFeb 16, 2021
  19. 3/5 commit-graph: use parse_options_concat()Ævar Arnfjörð Bjarmason, Feb 15, 2021
  20. Taylor BlauFeb 15, 2021
  21. Taylor BlauFeb 15, 2021
  22. Ævar Arnfjörð BjarmasonFeb 15, 2021
  23. SZEDER GáborSep 17, 2021
  24. Jeff KingSep 17, 2021
  25. Taylor BlauSep 18, 2021
  26. Ævar Arnfjörð BjarmasonSep 18, 2021
  27. Taylor BlauSep 18, 2021
  28. Taylor BlauSep 18, 2021
  29. Ævar Arnfjörð BjarmasonSep 18, 2021
  30. 5/5 commit-graph: show usage on "commit-graph [write|verify] garbage"Ævar Arnfjörð Bjarmason, Feb 15, 2021
  31. Taylor BlauFeb 15, 2021
  32. Derrick StoleeFeb 16, 2021
  33. 0/4 midx: split out sub-commandsTaylor Blau, Feb 15, 2021
  34. 1/4 builtin/multi-pack-index.c: inline 'flags' with optionsTaylor Blau, Feb 15, 2021
  35. 2/4 builtin/multi-pack-index.c: don't handle 'progress' separatelyTaylor Blau, Feb 15, 2021
  36. Ævar Arnfjörð BjarmasonFeb 15, 2021
  37. Taylor BlauFeb 15, 2021
  38. Derrick StoleeFeb 16, 2021
  39. 3/4 builtin/multi-pack-index.c: define common usage with a macroTaylor Blau, Feb 15, 2021
  40. 4/4 builtin/multi-pack-index.c: split sub-commandsTaylor Blau, Feb 15, 2021
  41. Ævar Arnfjörð BjarmasonFeb 15, 2021
  42. Taylor BlauFeb 15, 2021
  43. Ævar Arnfjörð BjarmasonFeb 15, 2021
  44. Taylor BlauFeb 15, 2021
  45. Derrick StoleeFeb 16, 2021
  46. Taylor BlauFeb 16, 2021
  47. 3/9 midx: don't free midx_name earlyTaylor Blau, Feb 10, 2021
  48. 4/9 midx: keep track of the checksumTaylor Blau, Feb 10, 2021
  49. Derrick StoleeFeb 11, 2021
  50. Taylor BlauFeb 11, 2021
  51. 5/9 midx: make some functions non-staticTaylor Blau, Feb 10, 2021
  52. 6/9 Documentation/technical: describe multi-pack reverse indexesTaylor Blau, Feb 10, 2021
  53. Derrick StoleeFeb 11, 2021
  54. Taylor BlauFeb 11, 2021
  55. 7/9 pack-revindex: read multi-pack reverse indexesTaylor Blau, Feb 10, 2021
  56. Derrick StoleeFeb 11, 2021
  57. Taylor BlauFeb 11, 2021
  58. Junio C HamanoFeb 11, 2021
  59. Taylor BlauFeb 11, 2021
  60. 8/9 pack-write.c: extract 'write_rev_file_order'Taylor Blau, Feb 10, 2021
  61. 9/9 pack-revindex: write multi-pack reverse indexesTaylor Blau, Feb 10, 2021
  62. Derrick StoleeFeb 11, 2021
  63. Taylor BlauFeb 11, 2021
  64. Junio C HamanoFeb 11, 2021
  65. Derrick StoleeFeb 11, 2021
  66. Junio C HamanoFeb 11, 2021
  67. 00/15 midx: implement a multi-pack reverse indexTaylor Blau, Feb 24, 2021
  68. 01/15 builtin/multi-pack-index.c: inline 'flags' with optionsTaylor Blau, Feb 24, 2021
  69. 03/15 builtin/multi-pack-index.c: define common usage with a macroTaylor Blau, Feb 24, 2021
  70. 02/15 builtin/multi-pack-index.c: don't handle 'progress' separatelyTaylor Blau, Feb 24, 2021
  71. 04/15 builtin/multi-pack-index.c: split sub-commandsTaylor Blau, Feb 24, 2021
  72. Jonathan TanMar 2, 2021
  73. Taylor BlauMar 2, 2021
  74. Jonathan TanMar 4, 2021
  75. Taylor BlauMar 4, 2021
  76. 06/15 builtin/multi-pack-index.c: display usage on unrecognized commandTaylor Blau, Feb 24, 2021
  77. 05/15 builtin/multi-pack-index.c: don't enter bogus cmd_modeTaylor Blau, Feb 24, 2021
  78. 07/15 t/helper/test-read-midx.c: add '--show-objects'Taylor Blau, Feb 24, 2021
  79. 08/15 midx: allow marking a pack as preferredTaylor Blau, Feb 24, 2021
  80. Jonathan TanMar 2, 2021
  81. Taylor BlauMar 2, 2021
  82. Jonathan TanMar 4, 2021
  83. Taylor BlauMar 4, 2021
  84. 15/15 pack-revindex: write multi-pack reverse indexesTaylor Blau, Feb 24, 2021
  85. Jonathan TanMar 2, 2021
  86. Taylor BlauMar 3, 2021
  87. Jonathan TanMar 4, 2021
  88. Taylor BlauMar 4, 2021
  89. 09/15 midx: don't free midx_name earlyTaylor Blau, Feb 24, 2021
  90. 11/15 midx: make some functions non-staticTaylor Blau, Feb 24, 2021
  91. 10/15 midx: keep track of the checksumTaylor Blau, Feb 24, 2021
  92. 12/15 Documentation/technical: describe multi-pack reverse indexesTaylor Blau, Feb 24, 2021
  93. Jonathan TanMar 2, 2021
  94. Taylor BlauMar 2, 2021
  95. Taylor BlauMar 2, 2021
  96. Jonathan TanMar 4, 2021
  97. 13/15 pack-revindex: read multi-pack reverse indexesTaylor Blau, Feb 24, 2021
  98. Jonathan TanMar 2, 2021
  99. Taylor BlauMar 3, 2021
  100. 14/15 pack-write.c: extract 'write_rev_file_order'Taylor Blau, Feb 24, 2021
  101. 00/16 midx: implement a multi-pack reverse indexTaylor Blau, Mar 11, 2021
  102. 02/16 builtin/multi-pack-index.c: don't handle 'progress' separatelyTaylor Blau, Mar 11, 2021
  103. Jeff KingMar 29, 2021
  104. 01/16 builtin/multi-pack-index.c: inline 'flags' with optionsTaylor Blau, Mar 11, 2021
  105. Jeff KingMar 29, 2021
  106. 03/16 builtin/multi-pack-index.c: define common usage with a macroTaylor Blau, Mar 11, 2021
  107. 04/16 builtin/multi-pack-index.c: split sub-commandsTaylor Blau, Mar 11, 2021
  108. Jeff KingMar 29, 2021
  109. Taylor BlauMar 29, 2021
  110. Jeff KingMar 30, 2021
  111. 05/16 builtin/multi-pack-index.c: don't enter bogus cmd_modeTaylor Blau, Mar 11, 2021
  112. 09/16 midx: don't free midx_name earlyTaylor Blau, Mar 11, 2021
  113. 07/16 t/helper/test-read-midx.c: add '--show-objects'Taylor Blau, Mar 11, 2021
  114. 06/16 builtin/multi-pack-index.c: display usage on unrecognized commandTaylor Blau, Mar 11, 2021
  115. Jeff KingMar 29, 2021
  116. Taylor BlauMar 29, 2021
  117. 08/16 midx: allow marking a pack as preferredTaylor Blau, Mar 11, 2021
  118. Jeff KingMar 29, 2021
  119. Taylor BlauMar 29, 2021
  120. Jeff KingMar 30, 2021
  121. 10/16 midx: keep track of the checksumTaylor Blau, Mar 11, 2021
  122. 11/16 midx: make some functions non-staticTaylor Blau, Mar 11, 2021
  123. 12/16 Documentation/technical: describe multi-pack reverse indexesTaylor Blau, Mar 11, 2021
  124. Jeff KingMar 29, 2021
  125. Taylor BlauMar 29, 2021
  126. 13/16 pack-revindex: read multi-pack reverse indexesTaylor Blau, Mar 11, 2021
  127. Jeff KingMar 29, 2021
  128. Taylor BlauMar 29, 2021
  129. 14/16 pack-write.c: extract 'write_rev_file_order'Taylor Blau, Mar 11, 2021
  130. 15/16 pack-revindex: write multi-pack reverse indexesTaylor Blau, Mar 11, 2021
  131. Jeff KingMar 29, 2021
  132. Taylor BlauMar 29, 2021
  133. 16/16 midx.c: improve cache locality in midx_pack_order_cmp()Taylor Blau, Mar 11, 2021
  134. Jeff KingMar 29, 2021
  135. Taylor BlauMar 29, 2021
  136. Jeff KingMar 30, 2021
  137. Derrick StoleeMar 12, 2021
  138. Jeff KingMar 29, 2021
  139. Junio C HamanoMar 29, 2021
  140. Taylor BlauMar 29, 2021
  141. Jeff KingMar 30, 2021
  142. Taylor BlauMar 30, 2021
  143. 00/16 midx: implement a multi-pack reverse indexTaylor Blau, Mar 30, 2021
  144. 01/16 builtin/multi-pack-index.c: inline 'flags' with optionsTaylor Blau, Mar 30, 2021
  145. 02/16 builtin/multi-pack-index.c: don't handle 'progress' separatelyTaylor Blau, Mar 30, 2021
  146. 04/16 builtin/multi-pack-index.c: split sub-commandsTaylor Blau, Mar 30, 2021
  147. 03/16 builtin/multi-pack-index.c: define common usage with a macroTaylor Blau, Mar 30, 2021
  148. 06/16 builtin/multi-pack-index.c: display usage on unrecognized commandTaylor Blau, Mar 30, 2021
  149. 05/16 builtin/multi-pack-index.c: don't enter bogus cmd_modeTaylor Blau, Mar 30, 2021
  150. 07/16 t/helper/test-read-midx.c: add '--show-objects'Taylor Blau, Mar 30, 2021
  151. 08/16 midx: allow marking a pack as preferredTaylor Blau, Mar 30, 2021
  152. Taylor BlauApr 1, 2021
  153. 10/16 midx: keep track of the checksumTaylor Blau, Mar 30, 2021
  154. 09/16 midx: don't free midx_name earlyTaylor Blau, Mar 30, 2021
  155. 11/16 midx: make some functions non-staticTaylor Blau, Mar 30, 2021
  156. 14/16 pack-write.c: extract 'write_rev_file_order'Taylor Blau, Mar 30, 2021
  157. pack-write: skip *.rev work when not writing *.revÆvar Arnfjörð Bjarmason, Sep 8, 2021
  158. Carlo ArenasSep 8, 2021
  159. Taylor BlauSep 8, 2021
  160. Junio C HamanoSep 8, 2021
  161. Taylor BlauSep 8, 2021
  162. Taylor BlauSep 8, 2021
  163. Ævar Arnfjörð BjarmasonSep 8, 2021
  164. Taylor BlauSep 8, 2021
  165. 12/16 Documentation/technical: describe multi-pack reverse indexesTaylor Blau, Mar 30, 2021
  166. 13/16 pack-revindex: read multi-pack reverse indexesTaylor Blau, Mar 30, 2021
  167. 16/16 midx.c: improve cache locality in midx_pack_order_cmp()Taylor Blau, Mar 30, 2021
  168. 15/16 pack-revindex: write multi-pack reverse indexesTaylor Blau, Mar 30, 2021
  169. Jeff KingMar 30, 2021
  170. Taylor BlauMar 30, 2021
  171. Jeff KingMar 30, 2021

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.