On Sun, 05 Jul 2020 at 01:13:25, Kevin Morris wrote:
Alright, patch sent; I used `-v1` this time with `git send-email`... I can't find documentation for that switch yet. I've tested basic search through both paths; the only refactoring that needed to be done was to remove the extra "AND (" and ")" from the generic function, and add it where we need it in `pkg_search_page`. Then, we can reuse the same `construct_keyword_search` in rpc.
Thanks for the revision! I added some comments inline. It's common to not use any versioning for the first revision of a patch (e.g. send without -v1) but then start using (-v2, -v3, ...) for followup revisions. The -v argument is documented in the git-format-patch(1) man page, if you are using git-format-patch and git-send-email separately, you need to specify the argument when formatting the patch (however, in most cases, you can also run `git format patch` directly).