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

Remove the Copy bound on AssetId #14158

Merged
paritytech-processbot merged 12 commits into from
May 23, 2023
Merged

Remove the Copy bound on AssetId #14158

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
fce3089
Remove the `Copy` bound on `AssetId`
koute May 16, 2023
a366c9e
Also relax the `Copy` bound in the assets pallet
koute May 16, 2023
b3a939b
Fix warnings on the newest nightly Rust
koute May 16, 2023
49d0b1a
Remove some unnecessary `clone()`s
koute May 16, 2023
4130ea7
Try to satisfy clippy
koute May 16, 2023
f836ca8
Merge remote-tracking branch 'origin/master' into master_asset_copy
May 16, 2023
289c057
Remove some more unnecessary `clone()`s
koute May 17, 2023
33406d9
Merge remote-tracking branch 'origin/master' into master_asset_copy
koute May 20, 2023
898f827
Merge remote-tracking branch 'origin/master' into master_asset_copy
koute May 22, 2023
5549ed5
Merge remote-tracking branch 'origin/master' into master_asset_copy
koute May 23, 2023
1a61e16
Add more `.clone()`s for newly merged code
koute May 23, 2023
edd005d
Also add `clone()`s in the benchmarks
koute May 23, 2023
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
Also add clone()s in the benchmarks
  • Loading branch information
@koute
koute committed May 23, 2023
commit edd005d4830d0301c813cff93a84a2a74611318c
6 changes: 3 additions & 3 deletions frame/nft-fractionalization/src/benchmarking.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ benchmarks! {
let collection = T::BenchmarkHelper::collection(0);
let nft = T::BenchmarkHelper::nft(0);
let (caller, caller_lookup) = mint_nft::<T>(nft);
}: _(SystemOrigin::Signed(caller.clone()), collection, nft, asset, caller_lookup, 1000u32.into())
}: _(SystemOrigin::Signed(caller.clone()), collection, nft, asset.clone(), caller_lookup, 1000u32.into())
verify {
assert_last_event::<T>(
Event::NftFractionalized {
Expand All @@ -115,11 +115,11 @@ benchmarks! {
SystemOrigin::Signed(caller.clone()).into(),
collection,
nft,
asset,
asset.clone(),
caller_lookup.clone(),
1000u32.into(),
)?;
}: _(SystemOrigin::Signed(caller.clone()), collection, nft, asset, caller_lookup)
}: _(SystemOrigin::Signed(caller.clone()), collection, nft, asset.clone(), caller_lookup)
verify {
assert_last_event::<T>(
Event::NftUnified {
Expand Down