ZheNing Hu <adlternative@xxxxxxxxx> writes: >> >> + else if (skip_prefix(start, "(path)", &p)) >> >> + write_name_to_buf(sb, data->pathname); >> >> These are just "values". >> ... >> >> + else if (skip_prefix(start, "(size)", &p)) >> >> + strbuf_addf(sb, "size: %u", sd->sd_size); >> >> + else if (skip_prefix(start, "(flags)", &p)) >> >> + strbuf_addf(sb, "flags: %x", data->ce->ce_flags); >> >> These are not. >> > ... If someone else > need them for some reason, we can add them back. If someone else needs to see "size:" printed in front of the value of sd_size member, we DO NOT HAVE TO DO ANYTHING! That someone else can write "--format=size: %(size)" to do so themselves.
