Subject: Re: [PATCH v2 04/11] object-file API: have write_object_file() take "enum object_type"

From: Junio C Hamano <[email-masked]>
Date: Fri, 04 Feb 2022 12:52:08 -0800

Ævar Arnfjörð Bjarmason  <[email-masked]> writes:

> diff --git a/builtin/notes.c b/builtin/notes.c
> index 05d60483e82..d81acaeaf78 100644
> --- a/builtin/notes.c
> +++ b/builtin/notes.c
> @@ -199,9 +199,8 @@ static void prepare_note_data(const struct object_id *object, struct note_data *
>  
>  static void write_note_data(struct note_data *d, struct object_id *oid)
>  {
> -	if (write_object_file(d->buf.buf, d->buf.len, blob_type, oid)) {
> +	if (write_object_file(d->buf.buf, d->buf.len, OBJ_BLOB, oid)) {
>  		int status = die_message(_("unable to write note object"));
> -
>  		if (d->edit_path)

Unrelated change here.  It would be OK if it were an improvement,
but it is not X-<.

Everything else in this change looked sensible.