Re: [PATCH v3 00/10] Filter combination
From: Junio C Hamano <hidden>
Date: 2019-06-14 19:50:54
Matthew DeVore [off-list ref] writes:
It has been a while since a sent a roll-up. Here are the changes since v2:
- Re-use more URL-encoding logic in strbuf.c
* This was partially achieved by changing the helper function to accept a
function that will indicate whether some character must be escaped.
- Re-use more URL-decoding logic in url.c
- changed the filter_spec strbuf to a string_list to avoid explicit
initialization
- Remove logic to "expand" tree:#k and tree:#m filter specs since there is no
server that supports tree:# but does not support tree:#k, as they were
implemented at the same time.Since the v2 of this topic, cc/list-objects-filter-wo-sparse-path was merged to the mainline before Git 2.22 was tagged. As we won't be merging this topic to any maintenance track anyway, it is probably a good time to rebase it on v2.22.0, to avoid unnecessary conflicts. Thanks.