On Tue, 11 Feb 2025 at 12:13, Amir Goldstein <amir73il@xxxxxxxxx> wrote: > What do you say about moving this comment outside the loop and leaving here > only: > > /* Should redirects/metacopy to lower layers be followed? */ > if ((nextmetacopy && !ofs->config.metacopy) || > (nextredirect && !ovl_redirect_follow(ofs))) > break; Nice idea, except it would break the next patch. I don't like the duplication either, maybe move nextmetacopy/nextredirect into ovl_lookup_data and create a helper? Thanks, Miklos
