Change the type_from_string() macro into a function and drop the support for passing len < 0. Support for len < 0 was added in fe8e3b71805 (Refactor type_from_string() to allow continuing after detecting an error, 2014-09-10), but no callers use that form. Let's drop it to simplify this, and in preparation for simplifying these even further. Even though the argument was changed from ssize_t to the unsigned size_t C is by design forgiving about passing -1 as an unsigned type (it's just an alias for "set all bits)", let's detect any outstanding in-flight callers passing a -1. Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> --- object.c | 13 ++++++++++--- object.h | 4 ++-- 2 files changed, 12 insertions(+), 5 deletions(-) diff --git a/object.c b/object.c index bad9e17f25..aae2a27e55 100644 --- a/object.c +++ b/object.c @@ -35,12 +35,12 @@ const char *type_name(unsigned int type) return object_type_strings[type]; } -int type_from_string_gently(const char *str, ssize_t len, int gentle) +int type_from_string_gently(const char *str, size_t len, int gentle) { int i; - if (len < 0) - len = strlen(str); + if (len == ~(size_t)0) + BUG("type-from-string-gently no longer allows unspecified length"); for (i = 1; i < ARRAY_SIZE(object_type_strings); i++) if (!strncmp(str, object_type_strings[i], len) && @@ -53,6 +53,13 @@ int type_from_string_gently(const char *str, ssize_t len, int gentle) die(_("invalid object type \"%.*s\""), (int)len, str); } +int type_from_string(const char *str) +{ + size_t len = strlen(str); + int ret = type_from_string_gently(str, len, 0); + return ret; +} + /* * Return a numerical hash value between 0 and n-1 for the object with * the specified sha1. n must be a power of 2. Please note that the diff --git a/object.h b/object.h index 59daadce21..f9d8f4d22b 100644 --- a/object.h +++ b/object.h @@ -93,8 +93,8 @@ struct object { }; const char *type_name(unsigned int type); -int type_from_string_gently(const char *str, ssize_t, int gentle); -#define type_from_string(str) type_from_string_gently(str, -1, 0) +int type_from_string_gently(const char *str, size_t, int gentle); +int type_from_string(const char *str); /* * Return the current number of buckets in the object hashmap. -- 2.31.1.723.ga5d7868e4a