Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> writes:
> Add a convenience function to wrap the xsnprintf() command that
> generates loose object headers. This code was copy/pasted in various
> parts of the codebase, let's define it in one place and re-use it from
> there.
> ...
> +/**
> + * format_loose_header() is a thin wrapper around s xsnprintf() that
The name should have "object" somewhere in it. Not all readers can
be expected to know that you meant "loose" to be an acceptable short
hand for "loose object".
That nit aside, I think it is a good idea to give people a common
helper function to call. I am undecided if it is a good idea to
make it take enum or "const char *"; most everybody should be able
to say
format_object_header(type_name(OBJ_COMMIT), ...)
just fine, so two variants might be overkill, just to allow
format_object_header(OBJ_COMMIT, ...)
and to forbid
format_object_header("connit", ...)
I dunno.
