Re: [PATCH v2 4/5] Always check `parse_tree*()`'s return value

Junio C Hamano
2024-02-22T17:05:44Z

Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:

>> If you happen to reroll for some reason, perhaps: s/Always/always/
>
> Unless I am missing something we only ask the part of a oneline after an
> initial "<something>:" to be downcased.
>
> At least all the "Merge branch [...]" commits are still capitalized and
> nobody complains ;-)

Of course. Merge commits and single-parent commits are very
different.

I suspect that the lack of " :" was what triggered the comment,
but this being more about the callers all over the place in the
code, it may be hard to say what "area" this belongs to. If I were
pressed to choose:

Subject: parse_tree*(): make callers always check return value

would probably be what I would choose, but I usually opt for going
the lazy way of not labelling the area at all ;-).