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

[RFC PATCH 3/5] ref-filter: add deltabase option

From
Olga Telezhnaya <olyatelezhnaya@gmail.com>
Date
Nov 9, 2018, 07:44 UTC
Message-ID
<01020166f76d84e0-d2e0c0ad-bb3d-46ab-8834-3760546086d4-000000@eu-west-1.amazonses.com>
In-Reply-To
<01020166f76d845f-1a02a31e-5094-4b27-974d-a23811066c58-000000@eu-west-1.amazonses.com>

Add new formatting option: deltabase. If the object is stored as a delta on-disk, this expands to the 40-hex sha1 of the delta base object. Otherwise, expands to the null sha1 (40 zeroes). We have same option in cat-file command. Hopefully, in the end I will remove formatting code from cat-file and reuse formatting parts from ref-filter.

Signed-off-by: Olga Telezhnaia <olyatelezhnaya@gmail.com>
---
 ref-filter.c | 17 ++++++++++++++++-
 1 file changed, 16 insertions(+), 1 deletion(-)
diff --git a/ref-filter.c b/ref-filter.c
index 8ba1a4e72f2c3..3cfe01a039f8b 100644
--- a/ref-filter.c
+++ b/ref-filter.c
@@ -246,6 +246,18 @@ static int objectsize_atom_parser(const struct ref_format *format, struct used_a
 	return 0;
 }
 
+static int deltabase_atom_parser(const struct ref_format *format, struct used_atom *atom,
+				 const char *arg, struct strbuf *err)
+{
+	if (arg)
+		return strbuf_addf_ret(err, -1, _("%%(deltabase) does not take arguments"));
+	if (*atom->name == '*')
+		oi_deref.info.delta_base_sha1 = oi_deref.delta_base_oid.hash;
+	else
+		oi.info.delta_base_sha1 = oi.delta_base_oid.hash;
+	return 0;
+}
+
 static int body_atom_parser(const struct ref_format *format, struct used_atom *atom,
 			    const char *arg, struct strbuf *err)
 {
@@ -437,6 +449,7 @@ static struct {
 	{ "objecttype", SOURCE_OTHER, FIELD_STR, objecttype_atom_parser },
 	{ "objectsize", SOURCE_OTHER, FIELD_ULONG, objectsize_atom_parser },
 	{ "objectname", SOURCE_OTHER, FIELD_STR, objectname_atom_parser },
+	{ "deltabase", SOURCE_OTHER, FIELD_STR, deltabase_atom_parser },
 	{ "tree", SOURCE_OBJ },
 	{ "parent", SOURCE_OBJ },
 	{ "numparent", SOURCE_OBJ, FIELD_ULONG },
@@ -888,7 +901,9 @@ static void grab_common_values(struct atom_value *val, int deref, struct expand_
 		} else if (!strcmp(name, "objectsize")) {
 			v->value = oi->size;
 			v->s = xstrfmt("%lu", oi->size);
-		} else if (deref)
+		} else if (!strcmp(name, "deltabase"))
+			v->s = xstrdup(oid_to_hex(&oi->delta_base_oid));
+		else if (deref)
 			grab_objectname(name, &oi->oid, v, &used_atom[i]);
 	}
 }

--
https://github.com/git/git/pull/552
Previous: Olga TelezhnayaNext: Olga Telezhnaya
Message 2 of 15 in “ref-filter: add objectsize:disk option”
  1. 1/5 ref-filter: add objectsize:disk optionOlga Telezhnaya, Nov 9, 2018
  2. 3/5 ref-filter: add deltabase optionOlga Telezhnaya, Nov 9, 2018
  3. 4/5 ref-filter: add tests for deltabaseOlga Telezhnaya, Nov 9, 2018
  4. 2/5 ref-filter: add tests for objectsize:diskOlga Telezhnaya, Nov 9, 2018
  5. 5/5 ref-filter: add docs for new optionsOlga Telezhnaya, Nov 9, 2018
  6. Junio C HamanoNov 12, 2018
  7. Junio C HamanoNov 12, 2018
  8. Johannes SchindelinNov 12, 2018
  9. Jeff KingNov 12, 2018
  10. Junio C HamanoNov 13, 2018
  11. Оля ТележнаяNov 20, 2018
  12. Junio C HamanoNov 21, 2018
  13. Johannes SchindelinNov 22, 2018
  14. Junio C HamanoNov 24, 2018
  15. Jeff KingNov 12, 2018

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.