Re: [PATCH v2 1/4] t/unit-tests: implement clar specific oid helper functions
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Feb 24, 2025, 17:55 UTC
- Message-ID
- <xmqqwmdf5is1.fsf@gitster.g>
- In-Reply-To
- <20250224152704.70289-2-kuforiji98@gmail.com>
Seyi Kuforiji <kuforiji98@gmail.com> writes:
Show 26 quoted lines
> -static int get_oid_arbitrary_hex_algop(const char *hex, struct object_id *oid,
> +static void cl_parse_oid(const char *hex, struct object_id *oid,
> const struct git_hash_algo *algop)
> {
> int ret;
> size_t sz = strlen(hex);
> struct strbuf buf = STRBUF_INIT;
>
> - if (!check(sz <= algop->hexsz)) {
> - test_msg("BUG: hex string (%s) bigger than maximum allowed (%lu)",
> - hex, (unsigned long)algop->hexsz);
> - return -1;
> - }
> + cl_assert(sz <= algop->hexsz);
>
> strbuf_add(&buf, hex, sz);
> strbuf_addchars(&buf, '0', algop->hexsz - sz);
>
> - ret = get_oid_hex_algop(buf.buf, oid, algop);
> - if (!check_int(ret, ==, 0))
> - test_msg("BUG: invalid hex input (%s) provided", hex);
> + cl_assert_equal_i(get_oid_hex_algop(buf.buf, oid, algop), 0);
>
> strbuf_release(&buf);
> - return ret;
> }As you are not returning "ret" and making the function void, you made "int ret" an unused variable that needs removing.
Thanks.