Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Optimize multilocation reanchoring logic in XCM #6301

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

Optimize multilocation reanchoring logic in XCM #6301

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
449fc9c
Optimize multilocation reanchoring
tonyalaribe Nov 16, 2022
763a72d
remove debug comments
tonyalaribe Nov 16, 2022
0d40bd5
cleanup tests for reanchoring
tonyalaribe Nov 16, 2022
8cdce27
switch to while loop
tonyalaribe Nov 16, 2022
1070422
remove the now unused inverted method
tonyalaribe Nov 17, 2022
e4d6359
workaround to prevent mutating self in the unlikely case of an overflow
tonyalaribe Nov 17, 2022
92a1760
switch to checked_At, to error when there's an overflow
tonyalaribe Nov 21, 2022
1706e28
Trigger CI
tonyalaribe Nov 22, 2022
59b949d
Merge remote-tracking branch 'origin/master' into aa/optimize-multilo…
Nov 22, 2022
36db35e
Merge remote-tracking branch 'origin/master' into aa/optimize-multilo…
Nov 23, 2022
f7af2b2
Merge remote-tracking branch 'origin/master' into aa/optimize-multilo…
Nov 25, 2022
a3d4898
Merge remote-tracking branch 'origin/master' into aa/optimize-multilo…
Dec 7, 2022
44c0d8b
Merge remote-tracking branch 'origin/master' into aa/optimize-multilo…
Dec 19, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
remove debug comments
  • Loading branch information
@tonyalaribe
tonyalaribe committed Nov 16, 2022
commit 763a72d6d2da742a93e1e6ea9395c8068810eabf
27 changes: 0 additions & 27 deletions xcm/src/v1/multilocation.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -378,34 +378,7 @@ impl MultiLocation {
log::error!("🔥 normalized-self ==> {:?}", self);
log::error!("🔥 normalized-target ==> {:?}", target);
self.parents = self.parent_count() + target.parent_count() + target.interior().len() as u8;
tonyalaribe marked this conversation as resolved.
Show resolved Hide resolved
// log::error!("🔥 normalized-self ==> {:?}", self);

// Add the ancestry to the self.
// Where self = ../../p1000
// ancestry = p2000/p50
// self = p2000/p50/p1000
// log::error!("\n");
// log::error!("🔥 self ==> {:?}", id);
// log::error!("🔥 ancestry ==> {:?}", ancestry);
// log::error!("🔥 target ==> {:?}", target);

// self.prepend_with_ancestry(ancestry.clone()).map_err(|_| ())?;
// log::error!("🔥 prependwith ==> {:?}", self);
// for _ in 0..target.parent_count() {
// let _ = self.take_first_interior();
// // nself = nselfO.unwrap(); // FIXME: don't unwrap
// }

// if target.interior().first() == self.interior().first() {
// target.take_first_interior();
// self.take_first_interior();
// // Do this in a loop
// }

// log::error!("🔥 pre=result ==> {:?}", self);
// self.parents = (self.parents.saturating_sub(target.parents))
// .saturating_add(target.interior().len() as u8);
// log::error!("🔥 target ==> {:?}", self);
log::error!("🔥 result ==> {:?}", self);

Ok(())
Expand Down