And I did't say it right. The execution path where dashdash_pos is 0 is coming from "annotate". Sorry for my confusion on the previous mail. On Fri, Nov 20, 2015 at 7:25 PM, Edmundo Carmona Antoranz <eantoranz@xxxxxxxxx> wrote: > On Tue, Nov 17, 2015 at 7:20 PM, Edmundo Carmona Antoranz > <eantoranz@xxxxxxxxx> wrote: >> + if (!revs.pending.nr && !file_exists(path)) >> + die_errno("cannot stat path '%s'", path); >> + > > I was wondering if I should only check the path that is coming from > "blame" (which is where I'm taking the check from) by checking > dashdash_pos: > > if (!dashdash_pos && !revs.pending.nr && !file_exists(path)) > die_errno("cannot stat path '%s'", path); > > So that we don't apply the check if we are coming from normal > dashdash_pos != 0 (and which didn't apply the filesystem check). > > Thanks in advance. -- 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