On 11/2/2020 3:43 PM, Elijah Newren wrote: > + /* > + * If mbase, side1, and side2 all match, we can resolve early. Even > + * if these are trees, there will be no renames or anything > + * underneath. > + */ > + if (side1_matches_mbase && side2_matches_mbase) { Here is a case where if you were not caring about renames you could prevent recursion here when "!renames && sides_match". Something to think about. > + /* mbase, side1, & side2 all match; use mbase as resolution */ > + setup_path_info(opt, &pi, dirname, info->pathlen, fullpath, > + names, names+0, mbase_null, 0, > + filemask, dirmask, 1); > + return mask; > + } > + -Stolee