After some discussion [1], we're convinced that the original approach for adding in just another pickaxe mode to find blobs was too hacky. So I went the less hacky way and did some refactoring first (patches 1-3), Then we'll have the new pickaxe mode to find blobs in patch 4. It grew slightly larger as it had issues with the setup (we neither have a regex nor a KWS to init) in this new world, so there are a few more lines in there. The last patch is just the cherry on the cake, helping to keep users sane by warning when they try to use different pickaxe modes at the same time. Thanks, Stefan [1] https://public-inbox.org/git/CAGZ79kaB0G9zetF6QtC45+ZGLM3gOsYWV7e+gkCe2yKOhb0Ssg@xxxxxxxxxxxxxx/ Stefan Beller (5): diff.h: Make pickaxe_opts an unsigned bit field diff: migrate diff_flags.pickaxe_ignore_case to a pickaxe_opts bit diff: introduce DIFF_PICKAXE_KINDS_MASK diffcore: add a pickaxe option to find a specific blob diff: properly error out when combining multiple pickaxe options Documentation/diff-options.txt | 10 +++++++++ builtin/log.c | 4 ++-- combine-diff.c | 2 +- diff.c | 35 +++++++++++++++++++++++++++--- diff.h | 13 ++++++++++-- diffcore-pickaxe.c | 48 ++++++++++++++++++++++++------------------ revision.c | 7 ++++-- 7 files changed, 89 insertions(+), 30 deletions(-) -- 2.15.1.620.gb9897f4670-goog
