Den fre 5 feb. 2021 kl 11:19 skrev Julian Foad <julianf...@apache.org>:
> (Ugh, sorry for the previous blank reply.) > > Daniel Sahlberg wrote: > > [...] Is it intentional to have both comments? [...] It would make it > easier to understand (at least for me) if it was a single comment. [...] > > > > - /* Iterate over each path with explicit mergeinfo added by the merge. > */ > > - /* Iterate over the paths in a parent-to-child order so that inherited > > - * mergeinfo is propagated consistently from each parent path to its > > - * children. (Issue #4862) */ > > + /* Iterate over the paths with explicit mergeinfo added by the merge > > + * in a parent-to-child order so that inherited mergeinfo is > propagated > > + * consistently from each parent path to its children. (Issue #4862) > */ > > I intended it to be read as one comment containing two sentences about the > same iteration. Feel free to adjust as you like, now I've confirmed you're > not missing any hidden meaning. I think it would be clearest to remove the > comment markers but keep as two sentences. > Like this? /* Iterate over each path with explicit mergeinfo added by the merge. * Iterate in a parent-to-child order so that inherited mergeinfo is propagated * consistently from each parent path to its children. (Issue #4862) */
