comparison rust/hg-core/src/revlog/index.rs @ 51227:e553cd209215

rust-index: avoid double negation in find_gca_candidates()

51226:83091c14058c

            for parent in self.parents(current_rev)? {
                if parent == NULL_REVISION {
                    continue;
                }
                let parent_seen = &seen[parent.0 as usize];
                if !poison {
                    // Without the `interesting` accounting, this block would
                    // be logically equivalent to: parent_seen |= current_seen
                    // The parent counts as interesting if it was not already
                    // known to be an ancestor (would already have counted)
                    if parent_seen.is_empty() {
                        seen[parent.0 as usize] = current_seen.clone();
                        interesting += 1;
                    } else if *parent_seen != current_seen {
                        seen[parent.0 as usize].union(&current_seen);
                    }
                } else {
                    // this block is logically equivalent to poisoning parent
                    // and counting it as non interesting if it
                    // has been seen before (hence counted then as interesting)
                    if !parent_seen.is_empty() && !parent_seen.is_poisoned() {
                        interesting -= 1;
                    }
                    seen[parent.0 as usize].poison();
                }
            }

            current_rev = Revision(current_rev.0 - 1);
        }

51227:e553cd209215

            for parent in self.parents(current_rev)? {
                if parent == NULL_REVISION {
                    continue;
                }
                let parent_seen = &seen[parent.0 as usize];
                if poison {
                    // this block is logically equivalent to poisoning parent
                    // and counting it as non interesting if it
                    // has been seen before (hence counted then as interesting)
                    if !parent_seen.is_empty() && !parent_seen.is_poisoned() {
                        interesting -= 1;
                    }
                    seen[parent.0 as usize].poison();
                } else {
                    // Without the `interesting` accounting, this block would
                    // be logically equivalent to: parent_seen |= current_seen
                    // The parent counts as interesting if it was not already
                    // known to be an ancestor (would already have counted)
                    if parent_seen.is_empty() {
                        seen[parent.0 as usize] = current_seen.clone();
                        interesting += 1;
                    } else if *parent_seen != current_seen {
                        seen[parent.0 as usize].union(&current_seen);
                    }
                }
            }

            current_rev = Revision(current_rev.0 - 1);
        }