Skip to content

Commit 04ab9d5

Browse files
leighmccullochleighmcculloch
committed
change contract method parameters from references to owned values
1 parent d5e76f8 commit 04ab9d5

File tree

3 files changed

+29
-29
lines changed

3 files changed

+29
-29
lines changed

‎tests-expanded/test_contract_data_tests.rs‎

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -136,15 +136,15 @@ impl soroban_sdk::testutils::ContractFunctionSet for Contract {
136136
__contract_fn_set_registry::call(func, env, args)
137137
}
138138
}
139-
impl<'i> Contract {
140-
pub fn put(e: Env, key: &Symbol, val: &Symbol) {
141-
e.storage().persistent().set(key, val)
139+
impl Contract {
140+
pub fn put(e: Env, key: Symbol, val: Symbol) {
141+
e.storage().persistent().set(&key, &val)
142142
}
143-
pub fn get(e: Env, key: &'i mut Symbol) -> Option<Symbol> {
144-
e.storage().persistent().get(key)
143+
pub fn get(e: Env, key: Symbol) -> Option<Symbol> {
144+
e.storage().persistent().get(&key)
145145
}
146-
pub fn del(e: Env, key: &Symbol) {
147-
e.storage().persistent().remove(key)
146+
pub fn del(e: Env, key: Symbol) {
147+
e.storage().persistent().remove(&key)
148148
}
149149
}
150150
#[doc(hidden)]
@@ -460,13 +460,13 @@ pub mod __Contract__put {
460460
#[allow(deprecated)]
461461
&<super::Contract>::put(
462462
env.clone(),
463-
&<_ as soroban_sdk::unwrap::UnwrapOptimized>::unwrap_optimized(
463+
<_ as soroban_sdk::unwrap::UnwrapOptimized>::unwrap_optimized(
464464
<_ as soroban_sdk::TryFromValForContractFn<
465465
soroban_sdk::Env,
466466
soroban_sdk::Val,
467467
>>::try_from_val_for_contract_fn(&env, &arg_0),
468468
),
469-
&<_ as soroban_sdk::unwrap::UnwrapOptimized>::unwrap_optimized(
469+
<_ as soroban_sdk::unwrap::UnwrapOptimized>::unwrap_optimized(
470470
<_ as soroban_sdk::TryFromValForContractFn<
471471
soroban_sdk::Env,
472472
soroban_sdk::Val,
@@ -510,7 +510,7 @@ pub mod __Contract__get {
510510
#[allow(deprecated)]
511511
&<super::Contract>::get(
512512
env.clone(),
513-
&mut <_ as soroban_sdk::unwrap::UnwrapOptimized>::unwrap_optimized(
513+
<_ as soroban_sdk::unwrap::UnwrapOptimized>::unwrap_optimized(
514514
<_ as soroban_sdk::TryFromValForContractFn<
515515
soroban_sdk::Env,
516516
soroban_sdk::Val,
@@ -551,7 +551,7 @@ pub mod __Contract__del {
551551
#[allow(deprecated)]
552552
&<super::Contract>::del(
553553
env.clone(),
554-
&<_ as soroban_sdk::unwrap::UnwrapOptimized>::unwrap_optimized(
554+
<_ as soroban_sdk::unwrap::UnwrapOptimized>::unwrap_optimized(
555555
<_ as soroban_sdk::TryFromValForContractFn<
556556
soroban_sdk::Env,
557557
soroban_sdk::Val,

‎tests-expanded/test_contract_data_wasm32v1-none.rs‎

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -24,15 +24,15 @@ impl<'a> ContractClient<'a> {
2424
}
2525
}
2626
}
27-
impl<'i> Contract {
28-
pub fn put(e: Env, key: &Symbol, val: &Symbol) {
29-
e.storage().persistent().set(key, val)
27+
impl Contract {
28+
pub fn put(e: Env, key: Symbol, val: Symbol) {
29+
e.storage().persistent().set(&key, &val)
3030
}
31-
pub fn get(e: Env, key: &'i mut Symbol) -> Option<Symbol> {
32-
e.storage().persistent().get(key)
31+
pub fn get(e: Env, key: Symbol) -> Option<Symbol> {
32+
e.storage().persistent().get(&key)
3333
}
34-
pub fn del(e: Env, key: &Symbol) {
35-
e.storage().persistent().remove(key)
34+
pub fn del(e: Env, key: Symbol) {
35+
e.storage().persistent().remove(&key)
3636
}
3737
}
3838
#[doc(hidden)]
@@ -222,13 +222,13 @@ pub mod __Contract__put {
222222
#[allow(deprecated)]
223223
&<super::Contract>::put(
224224
env.clone(),
225-
&<_ as soroban_sdk::unwrap::UnwrapOptimized>::unwrap_optimized(
225+
<_ as soroban_sdk::unwrap::UnwrapOptimized>::unwrap_optimized(
226226
<_ as soroban_sdk::TryFromValForContractFn<
227227
soroban_sdk::Env,
228228
soroban_sdk::Val,
229229
>>::try_from_val_for_contract_fn(&env, &arg_0),
230230
),
231-
&<_ as soroban_sdk::unwrap::UnwrapOptimized>::unwrap_optimized(
231+
<_ as soroban_sdk::unwrap::UnwrapOptimized>::unwrap_optimized(
232232
<_ as soroban_sdk::TryFromValForContractFn<
233233
soroban_sdk::Env,
234234
soroban_sdk::Val,
@@ -259,7 +259,7 @@ pub mod __Contract__get {
259259
#[allow(deprecated)]
260260
&<super::Contract>::get(
261261
env.clone(),
262-
&mut <_ as soroban_sdk::unwrap::UnwrapOptimized>::unwrap_optimized(
262+
<_ as soroban_sdk::unwrap::UnwrapOptimized>::unwrap_optimized(
263263
<_ as soroban_sdk::TryFromValForContractFn<
264264
soroban_sdk::Env,
265265
soroban_sdk::Val,
@@ -287,7 +287,7 @@ pub mod __Contract__del {
287287
#[allow(deprecated)]
288288
&<super::Contract>::del(
289289
env.clone(),
290-
&<_ as soroban_sdk::unwrap::UnwrapOptimized>::unwrap_optimized(
290+
<_ as soroban_sdk::unwrap::UnwrapOptimized>::unwrap_optimized(
291291
<_ as soroban_sdk::TryFromValForContractFn<
292292
soroban_sdk::Env,
293293
soroban_sdk::Val,

‎tests/contract_data/src/lib.rs‎

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -5,16 +5,16 @@ use soroban_sdk::{contract, contractimpl, Env, Symbol};
55
pub struct Contract;
66

77
#[contractimpl]
8-
impl<'i> Contract {
9-
pub fn put(e: Env, key: &Symbol, val: &Symbol) {
10-
e.storage().persistent().set(key, val)
8+
impl Contract {
9+
pub fn put(e: Env, key: Symbol, val: Symbol) {
10+
e.storage().persistent().set(&key, &val)
1111
}
1212

13-
pub fn get(e: Env, key: &'i mut Symbol) -> Option<Symbol> {
14-
e.storage().persistent().get(key)
13+
pub fn get(e: Env, key: Symbol) -> Option<Symbol> {
14+
e.storage().persistent().get(&key)
1515
}
1616

17-
pub fn del(e: Env, key: &Symbol) {
18-
e.storage().persistent().remove(key)
17+
pub fn del(e: Env, key: Symbol) {
18+
e.storage().persistent().remove(&key)
1919
}
2020
}

0 commit comments

Comments
 (0)