On Sun, Jun 21, 2015 at 1:48 PM, Karthik Nayak <karthik.188@xxxxxxxxx> wrote: > Rename parse_opt_with_commit() to parse_opt_commit_object_name() > to show that it can be used to obtain a list of commits and is > not constricted to usage of '--contains' option. I think that is a brilliant idea, but unlike the other function you added earlier that can do only one object and adopts "last one wins" rule, this is cumulative, and that fact should be made clear to the developers in some way, no? -- To unsubscribe from this list: send the line "unsubscribe git" in