Matheus Tavares Bernardino <matheus.bernardino@xxxxxx> writes: > On Tue, Jul 28, 2020 at 6:40 PM René Scharfe <l.s.r@xxxxxx> wrote: >> >> parse_object_or_die() is passed an object ID and a name to show if the >> object cannot be parsed. If the name is NULL then it shows the >> hexadecimal object ID. Use that feature instead of preparing and >> passing the hexadecimal representation to the function proactively. >> That's shorter and a bit more efficient. >> >> Signed-off-by: René Scharfe <l.s.r@xxxxxx> > > Good catch. And this seems to be the only place where we use > `parse_object_or_die(oid, oid_to_hex(oid))`. > > Reviewed-by: Matheus Tavares <matheus.bernardino@xxxxxx> Yup, makes sense. Thanks, both.