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

[PATCH v2 13/13] sha1_name: convert uses of 40 to GIT_SHA1_HEXSZ

From
brian m. carlson <sandals@crustytoothpaste.net>
Date
Jul 13, 2017, 23:49 UTC
Message-ID
<20170713234930.949612-14-sandals@crustytoothpaste.net>
In-Reply-To
<20170713234930.949612-1-sandals@crustytoothpaste.net>

There are several uses of the constant 40 in find_unique_abbrev_r. Convert them to GIT_SHA1_HEXSZ.

Signed-off-by: brian m. carlson <sandals@crustytoothpaste.net>
---
 sha1_name.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/sha1_name.c b/sha1_name.c
index b49303271e..8b7fd7e134 100644
--- a/sha1_name.c
+++ b/sha1_name.c
@@ -501,10 +501,10 @@ int find_unique_abbrev_r(char *hex, const unsigned char *sha1, int len)
 	}
 
 	sha1_to_hex_r(hex, sha1);
-	if (len == 40 || !len)
-		return 40;
+	if (len == GIT_SHA1_HEXSZ || !len)
+		return GIT_SHA1_HEXSZ;
 	exists = has_sha1_file(sha1);
-	while (len < 40) {
+	while (len < GIT_SHA1_HEXSZ) {
 		struct object_id oid_ret;
 		status = get_short_oid(hex, len, &oid_ret, GET_OID_QUIETLY);
 		if (exists
Previous: brian m. carlsonNext: Brandon Williams
Message 14 of 15 in “object_id part 9”
  1. 00/13 object_id part 9brian m. carlson, Jul 13, 2017
  2. 01/13 builtin/fsck: convert remaining caller of get_sha1 to object_idbrian m. carlson, Jul 13, 2017
  3. 02/13 builtin/merge-tree: convert remaining caller of get_sha1 to object_idbrian m. carlson, Jul 13, 2017
  4. 10/13 Convert remaining callers of get_sha1 to get_oid.brian m. carlson, Jul 13, 2017
  5. 05/13 sequencer: convert to struct object_idbrian m. carlson, Jul 13, 2017
  6. 07/13 bisect: convert bisect_checkout to struct object_idbrian m. carlson, Jul 13, 2017
  7. 11/13 sha1_name: convert get_sha1* to get_oid*brian m. carlson, Jul 13, 2017
  8. 09/13 builtin/verify-tag: convert to struct object_idbrian m. carlson, Jul 13, 2017
  9. 06/13 builtin/update_ref: convert to struct object_idbrian m. carlson, Jul 13, 2017
  10. 08/13 builtin/unpack-file: convert to struct object_idbrian m. carlson, Jul 13, 2017
  11. 03/13 submodule: convert submodule config lookup to use object_idbrian m. carlson, Jul 13, 2017
  12. 04/13 remote: convert struct push_cas to struct object_idbrian m. carlson, Jul 13, 2017
  13. 12/13 sha1_name: convert GET_SHA1* flags to GET_OID*brian m. carlson, Jul 13, 2017
  14. 13/13 sha1_name: convert uses of 40 to GIT_SHA1_HEXSZbrian m. carlson, Jul 13, 2017
  15. Brandon WilliamsJul 14, 2017

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.