Apparently, I fixed this and forgot to re-run format-patch, so I sent out the same patch the second time... My fault on that one. I've at least checked what I sent this time around, and it seems to match what's in my current tree. :) The second and third patches should be unmodified. Thanks for catching that, Stefan! On Wed, Nov 25, 2015 at 12:43 PM, Stefan Beller <sbeller@xxxxxxxxxx> wrote: > On Fri, Nov 13, 2015 at 4:46 PM, Doug Kelly <dougk.ff7@xxxxxxxxx> wrote: >> return "no corresponding .idx"; >> - case PACKDIR_FILE_IDX: >> + else if (seen_bits & PACKDIR_FILE_IDX && seen_bits ^ ~PACKDIR_FILE_PACK) > > Did you intend to use > (seen_bits & PACKDIR_FILE_IDX && !(seen_bits & PACKDIR_FILE_PACK)) > here? > > I was just looking at the state in peff/pu and it still has the xor > variant, which exposes more > than just the selected bit to the decision IIRC. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html