(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. - Julian
