copy_tracing.rs
680 lines
| 22.5 KiB
| application/rls-services+xml
|
RustLexer
Simon Sapin
|
r47414 | #[cfg(test)] | ||
#[macro_use] | ||||
mod tests_support; | ||||
#[cfg(test)] | ||||
mod tests; | ||||
r46587 | use crate::utils::hg_path::HgPath; | |||
r46556 | use crate::utils::hg_path::HgPathBuf; | |||
use crate::Revision; | ||||
r46764 | use crate::NULL_REVISION; | |||
r46556 | ||||
Simon Sapin
|
r47413 | use bytes_cast::{unaligned, BytesCast}; | ||
r46768 | use im_rc::ordmap::Entry; | |||
r46577 | use im_rc::ordmap::OrdMap; | |||
r47327 | use im_rc::OrdSet; | |||
r46577 | ||||
r46674 | use std::cmp::Ordering; | |||
r46556 | use std::collections::HashMap; | |||
pub type PathCopies = HashMap<HgPathBuf, HgPathBuf>; | ||||
r46766 | type PathToken = usize; | |||
r46745 | ||||
r47324 | #[derive(Clone, Debug)] | |||
r47312 | struct CopySource { | |||
r46556 | /// revision at which the copy information was added | |||
rev: Revision, | ||||
/// the copy source, (Set to None in case of deletion of the associated | ||||
/// key) | ||||
r46745 | path: Option<PathToken>, | |||
r47315 | /// a set of previous `CopySource.rev` value directly or indirectly | |||
/// overwritten by this one. | ||||
r47327 | overwritten: OrdSet<Revision>, | |||
r46556 | } | |||
r47314 | impl CopySource { | |||
/// create a new CopySource | ||||
/// | ||||
/// Use this when no previous copy source existed. | ||||
fn new(rev: Revision, path: Option<PathToken>) -> Self { | ||||
r47315 | Self { | |||
rev, | ||||
path, | ||||
r47327 | overwritten: OrdSet::new(), | |||
r47315 | } | |||
r47314 | } | |||
/// create a new CopySource from merging two others | ||||
/// | ||||
/// Use this when merging two InternalPathCopies requires active merging of | ||||
/// some entries. | ||||
fn new_from_merge(rev: Revision, winner: &Self, loser: &Self) -> Self { | ||||
r47327 | let mut overwritten = OrdSet::new(); | |||
r47315 | overwritten.extend(winner.overwritten.iter().copied()); | |||
overwritten.extend(loser.overwritten.iter().copied()); | ||||
overwritten.insert(winner.rev); | ||||
overwritten.insert(loser.rev); | ||||
r47314 | Self { | |||
rev, | ||||
path: winner.path, | ||||
r47315 | overwritten: overwritten, | |||
r47314 | } | |||
} | ||||
/// Update the value of a pre-existing CopySource | ||||
/// | ||||
/// Use this when recording copy information from parent → child edges | ||||
fn overwrite(&mut self, rev: Revision, path: Option<PathToken>) { | ||||
r47315 | self.overwritten.insert(self.rev); | |||
r47314 | self.rev = rev; | |||
self.path = path; | ||||
} | ||||
/// Mark pre-existing copy information as "dropped" by a file deletion | ||||
/// | ||||
/// Use this when recording copy information from parent → child edges | ||||
fn mark_delete(&mut self, rev: Revision) { | ||||
r47315 | self.overwritten.insert(self.rev); | |||
r47314 | self.rev = rev; | |||
self.path = None; | ||||
} | ||||
r47315 | ||||
r47323 | /// Mark pre-existing copy information as "dropped" by a file deletion | |||
/// | ||||
/// Use this when recording copy information from parent → child edges | ||||
fn mark_delete_with_pair(&mut self, rev: Revision, other: &Self) { | ||||
self.overwritten.insert(self.rev); | ||||
if other.rev != rev { | ||||
self.overwritten.insert(other.rev); | ||||
} | ||||
self.overwritten.extend(other.overwritten.iter().copied()); | ||||
self.rev = rev; | ||||
self.path = None; | ||||
} | ||||
r47315 | fn is_overwritten_by(&self, other: &Self) -> bool { | |||
other.overwritten.contains(&self.rev) | ||||
} | ||||
r47314 | } | |||
r47324 | // For the same "dest", content generated for a given revision will always be | |||
// the same. | ||||
impl PartialEq for CopySource { | ||||
fn eq(&self, other: &Self) -> bool { | ||||
#[cfg(debug_assertions)] | ||||
{ | ||||
if self.rev == other.rev { | ||||
debug_assert!(self.path == other.path); | ||||
debug_assert!(self.overwritten == other.overwritten); | ||||
} | ||||
} | ||||
self.rev == other.rev | ||||
} | ||||
} | ||||
r46556 | /// maps CopyDestination to Copy Source (+ a "timestamp" for the operation) | |||
r47312 | type InternalPathCopies = OrdMap<PathToken, CopySource>; | |||
r46556 | ||||
r46587 | /// Represent active changes that affect the copy tracing. | |||
enum Action<'a> { | ||||
/// The parent ? children edge is removing a file | ||||
/// | ||||
/// (actually, this could be the edge from the other parent, but it does | ||||
/// not matters) | ||||
Removed(&'a HgPath), | ||||
/// The parent ? children edge introduce copy information between (dest, | ||||
/// source) | ||||
r47320 | CopiedFromP1(&'a HgPath, &'a HgPath), | |||
CopiedFromP2(&'a HgPath, &'a HgPath), | ||||
r46587 | } | |||
r46589 | /// This express the possible "special" case we can get in a merge | |||
/// | ||||
/// See mercurial/metadata.py for details on these values. | ||||
#[derive(PartialEq)] | ||||
enum MergeCase { | ||||
/// Merged: file had history on both side that needed to be merged | ||||
Merged, | ||||
/// Salvaged: file was candidate for deletion, but survived the merge | ||||
Salvaged, | ||||
/// Normal: Not one of the two cases above | ||||
Normal, | ||||
} | ||||
r46674 | const COPY_MASK: u8 = 3; | |||
const P1_COPY: u8 = 2; | ||||
const P2_COPY: u8 = 3; | ||||
const ACTION_MASK: u8 = 28; | ||||
const REMOVED: u8 = 12; | ||||
const MERGED: u8 = 8; | ||||
const SALVAGED: u8 = 16; | ||||
Simon Sapin
|
r47413 | #[derive(BytesCast)] | ||
#[repr(C)] | ||||
struct ChangedFilesIndexEntry { | ||||
flags: u8, | ||||
r46674 | ||||
Simon Sapin
|
r47413 | /// Only the end position is stored. The start is at the end of the | ||
/// previous entry. | ||||
destination_path_end_position: unaligned::U32Be, | ||||
r46674 | ||||
Simon Sapin
|
r47413 | source_index_entry_position: unaligned::U32Be, | ||
} | ||||
fn _static_assert_size_of() { | ||||
let _ = std::mem::transmute::<ChangedFilesIndexEntry, [u8; 9]>; | ||||
} | ||||
r46674 | ||||
Simon Sapin
|
r47413 | /// Represents the files affected by a changeset. | ||
/// | ||||
/// This holds a subset of `mercurial.metadata.ChangingFiles` as we do not need | ||||
/// all the data categories tracked by it. | ||||
pub struct ChangedFiles<'a> { | ||||
index: &'a [ChangedFilesIndexEntry], | ||||
paths: &'a [u8], | ||||
} | ||||
r46674 | ||||
Simon Sapin
|
r47413 | impl<'a> ChangedFiles<'a> { | ||
pub fn new(data: &'a [u8]) -> Self { | ||||
let (header, rest) = unaligned::U32Be::from_bytes(data).unwrap(); | ||||
let nb_index_entries = header.get() as usize; | ||||
let (index, paths) = | ||||
ChangedFilesIndexEntry::slice_from_bytes(rest, nb_index_entries) | ||||
.unwrap(); | ||||
Self { index, paths } | ||||
r46556 | } | |||
pub fn new_empty() -> Self { | ||||
ChangedFiles { | ||||
Simon Sapin
|
r47413 | index: &[], | ||
paths: &[], | ||||
r46674 | } | |||
} | ||||
Simon Sapin
|
r47413 | /// Internal function to return the filename of the entry at a given index | ||
fn path(&self, idx: usize) -> &HgPath { | ||||
let start = if idx == 0 { | ||||
0 | ||||
r46674 | } else { | |||
Simon Sapin
|
r47413 | self.index[idx - 1].destination_path_end_position.get() as usize | ||
}; | ||||
let end = self.index[idx].destination_path_end_position.get() as usize; | ||||
HgPath::new(&self.paths[start..end]) | ||||
r46556 | } | |||
r46587 | ||||
/// Return an iterator over all the `Action` in this instance. | ||||
Simon Sapin
|
r47413 | fn iter_actions(&self) -> impl Iterator<Item = Action> { | ||
self.index.iter().enumerate().flat_map(move |(idx, entry)| { | ||||
let path = self.path(idx); | ||||
if (entry.flags & ACTION_MASK) == REMOVED { | ||||
Some(Action::Removed(path)) | ||||
} else if (entry.flags & COPY_MASK) == P1_COPY { | ||||
let source_idx = | ||||
entry.source_index_entry_position.get() as usize; | ||||
Some(Action::CopiedFromP1(path, self.path(source_idx))) | ||||
} else if (entry.flags & COPY_MASK) == P2_COPY { | ||||
let source_idx = | ||||
entry.source_index_entry_position.get() as usize; | ||||
Some(Action::CopiedFromP2(path, self.path(source_idx))) | ||||
} else { | ||||
None | ||||
} | ||||
}) | ||||
r46587 | } | |||
r46589 | ||||
/// return the MergeCase value associated with a filename | ||||
fn get_merge_case(&self, path: &HgPath) -> MergeCase { | ||||
Simon Sapin
|
r47413 | if self.index.is_empty() { | ||
r46589 | return MergeCase::Normal; | |||
} | ||||
r46674 | let mut low_part = 0; | |||
Simon Sapin
|
r47413 | let mut high_part = self.index.len(); | ||
r46674 | ||||
while low_part < high_part { | ||||
let cursor = (low_part + high_part - 1) / 2; | ||||
Simon Sapin
|
r47413 | match path.cmp(self.path(cursor)) { | ||
r46674 | Ordering::Less => low_part = cursor + 1, | |||
Ordering::Greater => high_part = cursor, | ||||
Ordering::Equal => { | ||||
Simon Sapin
|
r47413 | return match self.index[cursor].flags & ACTION_MASK { | ||
r46674 | MERGED => MergeCase::Merged, | |||
SALVAGED => MergeCase::Salvaged, | ||||
_ => MergeCase::Normal, | ||||
}; | ||||
} | ||||
} | ||||
} | ||||
MergeCase::Normal | ||||
r46589 | } | |||
r46556 | } | |||
r46766 | /// A small "tokenizer" responsible of turning full HgPath into lighter | |||
/// PathToken | ||||
/// | ||||
/// Dealing with small object, like integer is much faster, so HgPath input are | ||||
/// turned into integer "PathToken" and converted back in the end. | ||||
#[derive(Clone, Debug, Default)] | ||||
struct TwoWayPathMap { | ||||
token: HashMap<HgPathBuf, PathToken>, | ||||
path: Vec<HgPathBuf>, | ||||
} | ||||
impl TwoWayPathMap { | ||||
fn tokenize(&mut self, path: &HgPath) -> PathToken { | ||||
match self.token.get(path) { | ||||
Some(a) => *a, | ||||
None => { | ||||
let a = self.token.len(); | ||||
let buf = path.to_owned(); | ||||
self.path.push(buf.clone()); | ||||
self.token.insert(buf, a); | ||||
a | ||||
} | ||||
} | ||||
} | ||||
fn untokenize(&self, token: PathToken) -> &HgPathBuf { | ||||
Martin von Zweigbergk
|
r47363 | assert!(token < self.path.len(), "Unknown token: {}", token); | ||
r46766 | &self.path[token] | |||
} | ||||
} | ||||
r46556 | /// Same as mercurial.copies._combine_changeset_copies, but in Rust. | |||
Simon Sapin
|
r47329 | pub struct CombineChangesetCopies { | ||
all_copies: HashMap<Revision, InternalPathCopies>, | ||||
path_map: TwoWayPathMap, | ||||
children_count: HashMap<Revision, usize>, | ||||
} | ||||
r46556 | ||||
Simon Sapin
|
r47329 | impl CombineChangesetCopies { | ||
pub fn new(children_count: HashMap<Revision, usize>) -> Self { | ||||
Self { | ||||
all_copies: HashMap::new(), | ||||
path_map: TwoWayPathMap::default(), | ||||
children_count, | ||||
} | ||||
} | ||||
r46766 | ||||
Simon Sapin
|
r47329 | /// Combined the given `changes` data specific to `rev` with the data | ||
/// previously given for its parents (and transitively, its ancestors). | ||||
pub fn add_revision( | ||||
&mut self, | ||||
rev: Revision, | ||||
p1: Revision, | ||||
p2: Revision, | ||||
changes: ChangedFiles<'_>, | ||||
) { | ||||
Simon Sapin
|
r47356 | self.add_revision_inner(rev, p1, p2, changes.iter_actions(), |path| { | ||
changes.get_merge_case(path) | ||||
}) | ||||
} | ||||
/// Separated out from `add_revsion` so that unit tests can call this | ||||
/// without synthetizing a `ChangedFiles` in binary format. | ||||
fn add_revision_inner<'a>( | ||||
&mut self, | ||||
rev: Revision, | ||||
p1: Revision, | ||||
p2: Revision, | ||||
copy_actions: impl Iterator<Item = Action<'a>>, | ||||
get_merge_case: impl Fn(&HgPath) -> MergeCase + Copy, | ||||
) { | ||||
r47317 | // Retrieve data computed in a previous iteration | |||
let p1_copies = match p1 { | ||||
NULL_REVISION => None, | ||||
_ => get_and_clean_parent_copies( | ||||
Simon Sapin
|
r47329 | &mut self.all_copies, | ||
&mut self.children_count, | ||||
r46764 | p1, | |||
r47317 | ), // will be None if the vertex is not to be traversed | |||
}; | ||||
let p2_copies = match p2 { | ||||
NULL_REVISION => None, | ||||
_ => get_and_clean_parent_copies( | ||||
Simon Sapin
|
r47329 | &mut self.all_copies, | ||
&mut self.children_count, | ||||
r46764 | p2, | |||
r47317 | ), // will be None if the vertex is not to be traversed | |||
}; | ||||
r47318 | // combine it with data for that revision | |||
Simon Sapin
|
r47329 | let (p1_copies, p2_copies) = chain_changes( | ||
&mut self.path_map, | ||||
p1_copies, | ||||
p2_copies, | ||||
Simon Sapin
|
r47356 | copy_actions, | ||
Simon Sapin
|
r47329 | rev, | ||
); | ||||
r47318 | let copies = match (p1_copies, p2_copies) { | |||
(None, None) => None, | ||||
(c, None) => c, | ||||
(None, c) => c, | ||||
(Some(p1_copies), Some(p2_copies)) => Some(merge_copies_dict( | ||||
Simon Sapin
|
r47329 | &self.path_map, | ||
rev, | ||||
p2_copies, | ||||
p1_copies, | ||||
Simon Sapin
|
r47356 | get_merge_case, | ||
r47318 | )), | |||
}; | ||||
if let Some(c) = copies { | ||||
Simon Sapin
|
r47329 | self.all_copies.insert(rev, c); | ||
r46556 | } | |||
} | ||||
Simon Sapin
|
r47329 | /// Drop intermediate data (such as which revision a copy was from) and | ||
/// return the final mapping. | ||||
pub fn finish(mut self, target_rev: Revision) -> PathCopies { | ||||
let tt_result = self | ||||
.all_copies | ||||
.remove(&target_rev) | ||||
.expect("target revision was not processed"); | ||||
let mut result = PathCopies::default(); | ||||
for (dest, tt_source) in tt_result { | ||||
if let Some(path) = tt_source.path { | ||||
let path_dest = self.path_map.untokenize(dest).to_owned(); | ||||
let path_path = self.path_map.untokenize(path).to_owned(); | ||||
result.insert(path_dest, path_path); | ||||
} | ||||
r46556 | } | |||
Simon Sapin
|
r47329 | result | ||
r46556 | } | |||
} | ||||
r46764 | /// fetch previous computed information | |||
/// | ||||
/// If no other children are expected to need this information, we drop it from | ||||
/// the cache. | ||||
/// | ||||
/// If parent is not part of the set we are expected to walk, return None. | ||||
fn get_and_clean_parent_copies( | ||||
r47311 | all_copies: &mut HashMap<Revision, InternalPathCopies>, | |||
r46764 | children_count: &mut HashMap<Revision, usize>, | |||
parent_rev: Revision, | ||||
r47311 | ) -> Option<InternalPathCopies> { | |||
r46764 | let count = children_count.get_mut(&parent_rev)?; | |||
*count -= 1; | ||||
if *count == 0 { | ||||
match all_copies.remove(&parent_rev) { | ||||
Some(c) => Some(c), | ||||
r47311 | None => Some(InternalPathCopies::default()), | |||
r46764 | } | |||
} else { | ||||
match all_copies.get(&parent_rev) { | ||||
Some(c) => Some(c.clone()), | ||||
r47311 | None => Some(InternalPathCopies::default()), | |||
r46764 | } | |||
} | ||||
} | ||||
r46676 | /// Combine ChangedFiles with some existing PathCopies information and return | |||
/// the result | ||||
Simon Sapin
|
r47356 | fn chain_changes<'a>( | ||
r46766 | path_map: &mut TwoWayPathMap, | |||
r47321 | base_p1_copies: Option<InternalPathCopies>, | |||
base_p2_copies: Option<InternalPathCopies>, | ||||
Simon Sapin
|
r47356 | copy_actions: impl Iterator<Item = Action<'a>>, | ||
r46676 | current_rev: Revision, | |||
r47321 | ) -> (Option<InternalPathCopies>, Option<InternalPathCopies>) { | |||
// Fast path the "nothing to do" case. | ||||
if let (None, None) = (&base_p1_copies, &base_p2_copies) { | ||||
return (None, None); | ||||
} | ||||
let mut p1_copies = base_p1_copies.clone(); | ||||
let mut p2_copies = base_p2_copies.clone(); | ||||
Simon Sapin
|
r47356 | for action in copy_actions { | ||
r46676 | match action { | |||
r47320 | Action::CopiedFromP1(path_dest, path_source) => { | |||
r47321 | match &mut p1_copies { | |||
None => (), // This is not a vertex we should proceed. | ||||
Some(copies) => add_one_copy( | ||||
r47320 | current_rev, | |||
path_map, | ||||
r47321 | copies, | |||
base_p1_copies.as_ref().unwrap(), | ||||
r47320 | path_dest, | |||
path_source, | ||||
), | ||||
r47321 | } | |||
r47320 | } | |||
Action::CopiedFromP2(path_dest, path_source) => { | ||||
r47321 | match &mut p2_copies { | |||
None => (), // This is not a vertex we should proceed. | ||||
Some(copies) => add_one_copy( | ||||
r47320 | current_rev, | |||
path_map, | ||||
r47321 | copies, | |||
base_p2_copies.as_ref().unwrap(), | ||||
r47320 | path_dest, | |||
path_source, | ||||
), | ||||
r47321 | } | |||
r46676 | } | |||
r46766 | Action::Removed(deleted_path) => { | |||
r46676 | // We must drop copy information for removed file. | |||
// | ||||
// We need to explicitly record them as dropped to | ||||
// propagate this information when merging two | ||||
r47311 | // InternalPathCopies object. | |||
r46766 | let deleted = path_map.tokenize(deleted_path); | |||
r47322 | ||||
let p1_entry = match &mut p1_copies { | ||||
None => None, | ||||
Some(copies) => match copies.entry(deleted) { | ||||
Entry::Occupied(e) => Some(e), | ||||
Entry::Vacant(_) => None, | ||||
}, | ||||
}; | ||||
let p2_entry = match &mut p2_copies { | ||||
None => None, | ||||
Some(copies) => match copies.entry(deleted) { | ||||
Entry::Occupied(e) => Some(e), | ||||
Entry::Vacant(_) => None, | ||||
}, | ||||
r47321 | }; | |||
r47322 | ||||
match (p1_entry, p2_entry) { | ||||
(None, None) => (), | ||||
(Some(mut e), None) => { | ||||
e.get_mut().mark_delete(current_rev) | ||||
r47321 | } | |||
r47322 | (None, Some(mut e)) => { | |||
e.get_mut().mark_delete(current_rev) | ||||
} | ||||
(Some(mut e1), Some(mut e2)) => { | ||||
r47323 | let cs1 = e1.get_mut(); | |||
let cs2 = e2.get(); | ||||
r47326 | if cs1 == cs2 { | |||
cs1.mark_delete(current_rev); | ||||
} else { | ||||
cs1.mark_delete_with_pair(current_rev, &cs2); | ||||
} | ||||
r47323 | e2.insert(cs1.clone()); | |||
r47322 | } | |||
} | ||||
r46676 | } | |||
} | ||||
} | ||||
r47321 | (p1_copies, p2_copies) | |||
r46676 | } | |||
r47319 | // insert one new copy information in an InternalPathCopies | |||
// | ||||
// This deal with chaining and overwrite. | ||||
fn add_one_copy( | ||||
current_rev: Revision, | ||||
path_map: &mut TwoWayPathMap, | ||||
copies: &mut InternalPathCopies, | ||||
base_copies: &InternalPathCopies, | ||||
path_dest: &HgPath, | ||||
path_source: &HgPath, | ||||
) { | ||||
let dest = path_map.tokenize(path_dest); | ||||
let source = path_map.tokenize(path_source); | ||||
let entry; | ||||
if let Some(v) = base_copies.get(&source) { | ||||
entry = match &v.path { | ||||
Some(path) => Some((*(path)).to_owned()), | ||||
None => Some(source.to_owned()), | ||||
} | ||||
} else { | ||||
entry = Some(source.to_owned()); | ||||
} | ||||
// Each new entry is introduced by the children, we | ||||
// record this information as we will need it to take | ||||
// the right decision when merging conflicting copy | ||||
// information. See merge_copies_dict for details. | ||||
match copies.entry(dest) { | ||||
Entry::Vacant(slot) => { | ||||
let ttpc = CopySource::new(current_rev, entry); | ||||
slot.insert(ttpc); | ||||
} | ||||
Entry::Occupied(mut slot) => { | ||||
let ttpc = slot.get_mut(); | ||||
ttpc.overwrite(current_rev, entry); | ||||
} | ||||
} | ||||
} | ||||
r46556 | /// merge two copies-mapping together, minor and major | |||
/// | ||||
/// In case of conflict, value from "major" will be picked, unless in some | ||||
/// cases. See inline documentation for details. | ||||
r47316 | fn merge_copies_dict( | |||
r46766 | path_map: &TwoWayPathMap, | |||
r46771 | current_merge: Revision, | |||
Simon Sapin
|
r47328 | minor: InternalPathCopies, | ||
major: InternalPathCopies, | ||||
Simon Sapin
|
r47356 | get_merge_case: impl Fn(&HgPath) -> MergeCase + Copy, | ||
r47311 | ) -> InternalPathCopies { | |||
Simon Sapin
|
r47328 | use crate::utils::{ordmap_union_with_merge, MergeResult}; | ||
Simon Sapin
|
r47355 | ordmap_union_with_merge(minor, major, |&dest, src_minor, src_major| { | ||
Simon Sapin
|
r47328 | let (pick, overwrite) = compare_value( | ||
current_merge, | ||||
Simon Sapin
|
r47356 | || get_merge_case(path_map.untokenize(dest)), | ||
Simon Sapin
|
r47328 | src_minor, | ||
src_major, | ||||
); | ||||
if overwrite { | ||||
let (winner, loser) = match pick { | ||||
MergePick::Major | MergePick::Any => (src_major, src_minor), | ||||
MergePick::Minor => (src_minor, src_major), | ||||
}; | ||||
MergeResult::UseNewValue(CopySource::new_from_merge( | ||||
r47316 | current_merge, | |||
Simon Sapin
|
r47328 | winner, | ||
loser, | ||||
)) | ||||
} else { | ||||
match pick { | ||||
MergePick::Any | MergePick::Major => { | ||||
MergeResult::UseRightValue | ||||
r47310 | } | |||
Simon Sapin
|
r47328 | MergePick::Minor => MergeResult::UseLeftValue, | ||
r46742 | } | |||
r46556 | } | |||
Simon Sapin
|
r47328 | }) | ||
r46556 | } | |||
r46741 | ||||
/// represent the side that should prevail when merging two | ||||
r47311 | /// InternalPathCopies | |||
Simon Sapin
|
r47356 | #[derive(Debug, PartialEq)] | ||
r46741 | enum MergePick { | |||
/// The "major" (p1) side prevails | ||||
Major, | ||||
/// The "minor" (p2) side prevails | ||||
Minor, | ||||
/// Any side could be used (because they are the same) | ||||
Any, | ||||
} | ||||
/// decide which side prevails in case of conflicting values | ||||
#[allow(clippy::if_same_then_else)] | ||||
r47316 | fn compare_value( | |||
r46771 | current_merge: Revision, | |||
Simon Sapin
|
r47356 | merge_case_for_dest: impl Fn() -> MergeCase, | ||
r47312 | src_minor: &CopySource, | |||
src_major: &CopySource, | ||||
r47310 | ) -> (MergePick, bool) { | |||
r47325 | if src_major == src_minor { | |||
(MergePick::Any, false) | ||||
} else if src_major.rev == current_merge { | ||||
// minor is different according to per minor == major check earlier | ||||
debug_assert!(src_minor.rev != current_merge); | ||||
// The last value comes the current merge, this value -will- win | ||||
// eventually. | ||||
(MergePick::Major, true) | ||||
r46771 | } else if src_minor.rev == current_merge { | |||
// The last value comes the current merge, this value -will- win | ||||
// eventually. | ||||
r47310 | (MergePick::Minor, true) | |||
r46771 | } else if src_major.path == src_minor.path { | |||
r47325 | debug_assert!(src_major.rev != src_major.rev); | |||
r46741 | // we have the same value, but from other source; | |||
r47325 | if src_major.is_overwritten_by(src_minor) { | |||
r47310 | (MergePick::Minor, false) | |||
r47315 | } else if src_minor.is_overwritten_by(src_major) { | |||
r47310 | (MergePick::Major, false) | |||
r46741 | } else { | |||
r47310 | (MergePick::Any, true) | |||
r46741 | } | |||
} else { | ||||
r47325 | debug_assert!(src_major.rev != src_major.rev); | |||
Simon Sapin
|
r47356 | let action = merge_case_for_dest(); | ||
r47309 | if src_minor.path.is_some() | |||
&& src_major.path.is_none() | ||||
&& action == MergeCase::Salvaged | ||||
{ | ||||
r46741 | // If the file is "deleted" in the major side but was | |||
// salvaged by the merge, we keep the minor side alive | ||||
r47310 | (MergePick::Minor, true) | |||
r47309 | } else if src_major.path.is_some() | |||
&& src_minor.path.is_none() | ||||
&& action == MergeCase::Salvaged | ||||
{ | ||||
r46741 | // If the file is "deleted" in the minor side but was | |||
// salvaged by the merge, unconditionnaly preserve the | ||||
// major side. | ||||
r47310 | (MergePick::Major, true) | |||
r47315 | } else if src_minor.is_overwritten_by(src_major) { | |||
r47309 | // The information from the minor version are strictly older than | |||
// the major version | ||||
if action == MergeCase::Merged { | ||||
// If the file was actively merged, its means some non-copy | ||||
// activity happened on the other branch. It | ||||
// mean the older copy information are still relevant. | ||||
// | ||||
// The major side wins such conflict. | ||||
r47310 | (MergePick::Major, true) | |||
r47309 | } else { | |||
// No activity on the minor branch, pick the newer one. | ||||
r47310 | (MergePick::Major, false) | |||
r47309 | } | |||
r47315 | } else if src_major.is_overwritten_by(src_minor) { | |||
r47309 | if action == MergeCase::Merged { | |||
// If the file was actively merged, its means some non-copy | ||||
// activity happened on the other branch. It | ||||
// mean the older copy information are still relevant. | ||||
// | ||||
// The major side wins such conflict. | ||||
r47310 | (MergePick::Major, true) | |||
r47309 | } else { | |||
// No activity on the minor branch, pick the newer one. | ||||
r47310 | (MergePick::Minor, false) | |||
r47309 | } | |||
} else if src_minor.path.is_none() { | ||||
// the minor side has no relevant information, pick the alive one | ||||
r47310 | (MergePick::Major, true) | |||
r47309 | } else if src_major.path.is_none() { | |||
// the major side has no relevant information, pick the alive one | ||||
r47310 | (MergePick::Minor, true) | |||
r46741 | } else { | |||
r47309 | // by default the major side wins | |||
r47310 | (MergePick::Major, true) | |||
r46741 | } | |||
} | ||||
} | ||||