Minor renaming diff -> state_diff
This commit is contained in:
parent
bc5c3da2af
commit
b9ec87548d
@ -464,9 +464,9 @@ impl<V> BlockChainClient for Client<V> where V: Verifier {
|
|||||||
let options = TransactOptions { tracing: false, vm_tracing: analytics.vm_tracing, check_nonce: false };
|
let options = TransactOptions { tracing: false, vm_tracing: analytics.vm_tracing, check_nonce: false };
|
||||||
let mut ret = Executive::new(&mut state, &env_info, self.engine.deref().deref(), &self.vm_factory).transact(t, options);
|
let mut ret = Executive::new(&mut state, &env_info, self.engine.deref().deref(), &self.vm_factory).transact(t, options);
|
||||||
// TODO gav move this into Executive.
|
// TODO gav move this into Executive.
|
||||||
if analytics.diffing {
|
if analytics.state_diffing {
|
||||||
if let Ok(ref mut x) = ret {
|
if let Ok(ref mut x) = ret {
|
||||||
x.diff = Some(state.diff_from(self.state()));
|
x.state_diff = Some(state.diff_from(self.state()));
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
ret
|
ret
|
||||||
|
@ -55,7 +55,7 @@ pub struct CallAnalytics {
|
|||||||
/// Make a VM trace.
|
/// Make a VM trace.
|
||||||
pub vm_tracing: bool,
|
pub vm_tracing: bool,
|
||||||
/// Make a diff.
|
/// Make a diff.
|
||||||
pub diffing: bool,
|
pub state_diffing: bool,
|
||||||
}
|
}
|
||||||
|
|
||||||
/// Blockchain database client. Owns and manages a blockchain and a block queue.
|
/// Blockchain database client. Owns and manages a blockchain and a block queue.
|
||||||
|
@ -450,7 +450,7 @@ impl<'a> Executive<'a> {
|
|||||||
output: output,
|
output: output,
|
||||||
trace: trace,
|
trace: trace,
|
||||||
vm_trace: vm_trace,
|
vm_trace: vm_trace,
|
||||||
diff: None,
|
state_diff: None,
|
||||||
})
|
})
|
||||||
},
|
},
|
||||||
_ => {
|
_ => {
|
||||||
@ -464,7 +464,7 @@ impl<'a> Executive<'a> {
|
|||||||
output: output,
|
output: output,
|
||||||
trace: trace,
|
trace: trace,
|
||||||
vm_trace: vm_trace,
|
vm_trace: vm_trace,
|
||||||
diff: None,
|
state_diff: None,
|
||||||
})
|
})
|
||||||
},
|
},
|
||||||
}
|
}
|
||||||
|
@ -280,9 +280,9 @@ impl MinerService for Miner {
|
|||||||
|
|
||||||
let mut ret = Executive::new(&mut state, &env_info, self.engine(), chain.vm_factory()).transact(t, options);
|
let mut ret = Executive::new(&mut state, &env_info, self.engine(), chain.vm_factory()).transact(t, options);
|
||||||
// TODO gav move this into Executive.
|
// TODO gav move this into Executive.
|
||||||
if analytics.diffing {
|
if analytics.state_diffing {
|
||||||
if let Ok(ref mut x) = ret {
|
if let Ok(ref mut x) = ret {
|
||||||
x.diff = Some(state.diff_from(block.state().clone()));
|
x.state_diff = Some(state.diff_from(block.state().clone()));
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
ret
|
ret
|
||||||
|
@ -63,7 +63,7 @@ pub struct Executed {
|
|||||||
/// The VM trace of this transaction.
|
/// The VM trace of this transaction.
|
||||||
pub vm_trace: Option<VMTrace>,
|
pub vm_trace: Option<VMTrace>,
|
||||||
/// The state diff, if we traced it.
|
/// The state diff, if we traced it.
|
||||||
pub diff: Option<StateDiff>,
|
pub state_diff: Option<StateDiff>,
|
||||||
}
|
}
|
||||||
|
|
||||||
/// Result of executing the transaction.
|
/// Result of executing the transaction.
|
||||||
|
@ -255,7 +255,7 @@ impl<C, M> Ethcore for EthcoreClient<C, M> where C: BlockChainClient + 'static,
|
|||||||
from_params(params)
|
from_params(params)
|
||||||
.and_then(|(request,)| {
|
.and_then(|(request,)| {
|
||||||
let signed = try!(self.sign_call(request));
|
let signed = try!(self.sign_call(request));
|
||||||
let r = take_weak!(self.client).call(&signed, CallAnalytics{ vm_tracing: true, diffing: false });
|
let r = take_weak!(self.client).call(&signed, CallAnalytics{ vm_tracing: true, state_diffing: false });
|
||||||
if let Ok(executed) = r {
|
if let Ok(executed) = r {
|
||||||
if let Some(vm_trace) = executed.vm_trace {
|
if let Some(vm_trace) = executed.vm_trace {
|
||||||
return Ok(vm_trace_to_object(&vm_trace));
|
return Ok(vm_trace_to_object(&vm_trace));
|
||||||
@ -265,14 +265,14 @@ impl<C, M> Ethcore for EthcoreClient<C, M> where C: BlockChainClient + 'static,
|
|||||||
})
|
})
|
||||||
}
|
}
|
||||||
|
|
||||||
fn diff_call(&self, params: Params) -> Result<Value, Error> {
|
fn state_diff_call(&self, params: Params) -> Result<Value, Error> {
|
||||||
trace!(target: "jsonrpc", "diff_call: {:?}", params);
|
trace!(target: "jsonrpc", "state_diff_call: {:?}", params);
|
||||||
from_params(params)
|
from_params(params)
|
||||||
.and_then(|(request,)| {
|
.and_then(|(request,)| {
|
||||||
let signed = try!(self.sign_call(request));
|
let signed = try!(self.sign_call(request));
|
||||||
let r = take_weak!(self.client).call(&signed, CallAnalytics{ vm_tracing: false, diffing: true });
|
let r = take_weak!(self.client).call(&signed, CallAnalytics{ vm_tracing: false, state_diffing: true });
|
||||||
if let Ok(executed) = r {
|
if let Ok(executed) = r {
|
||||||
if let Some(state_diff) = executed.diff {
|
if let Some(state_diff) = executed.state_diff {
|
||||||
return Ok(state_diff_to_object(&state_diff));
|
return Ok(state_diff_to_object(&state_diff));
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -431,7 +431,7 @@ fn rpc_eth_call() {
|
|||||||
output: vec![0x12, 0x34, 0xff],
|
output: vec![0x12, 0x34, 0xff],
|
||||||
trace: None,
|
trace: None,
|
||||||
vm_trace: None,
|
vm_trace: None,
|
||||||
diff: None,
|
state_diff: None,
|
||||||
});
|
});
|
||||||
|
|
||||||
let request = r#"{
|
let request = r#"{
|
||||||
@ -466,7 +466,7 @@ fn rpc_eth_call_default_block() {
|
|||||||
output: vec![0x12, 0x34, 0xff],
|
output: vec![0x12, 0x34, 0xff],
|
||||||
trace: None,
|
trace: None,
|
||||||
vm_trace: None,
|
vm_trace: None,
|
||||||
diff: None,
|
state_diff: None,
|
||||||
});
|
});
|
||||||
|
|
||||||
let request = r#"{
|
let request = r#"{
|
||||||
@ -500,7 +500,7 @@ fn rpc_eth_estimate_gas() {
|
|||||||
output: vec![0x12, 0x34, 0xff],
|
output: vec![0x12, 0x34, 0xff],
|
||||||
trace: None,
|
trace: None,
|
||||||
vm_trace: None,
|
vm_trace: None,
|
||||||
diff: None,
|
state_diff: None,
|
||||||
});
|
});
|
||||||
|
|
||||||
let request = r#"{
|
let request = r#"{
|
||||||
@ -535,7 +535,7 @@ fn rpc_eth_estimate_gas_default_block() {
|
|||||||
output: vec![0x12, 0x34, 0xff],
|
output: vec![0x12, 0x34, 0xff],
|
||||||
trace: None,
|
trace: None,
|
||||||
vm_trace: None,
|
vm_trace: None,
|
||||||
diff: None,
|
state_diff: None,
|
||||||
});
|
});
|
||||||
|
|
||||||
let request = r#"{
|
let request = r#"{
|
||||||
|
@ -76,7 +76,7 @@ pub trait Ethcore: Sized + Send + Sync + 'static {
|
|||||||
fn vm_trace_call(&self, _: Params) -> Result<Value, Error>;
|
fn vm_trace_call(&self, _: Params) -> Result<Value, Error>;
|
||||||
|
|
||||||
/// Executes the given call and returns the diff for it.
|
/// Executes the given call and returns the diff for it.
|
||||||
fn diff_call(&self, params: Params) -> Result<Value, Error>;
|
fn state_diff_call(&self, params: Params) -> Result<Value, Error>;
|
||||||
|
|
||||||
/// Should be used to convert object to io delegate.
|
/// Should be used to convert object to io delegate.
|
||||||
fn to_delegate(self) -> IoDelegate<Self> {
|
fn to_delegate(self) -> IoDelegate<Self> {
|
||||||
@ -101,7 +101,7 @@ pub trait Ethcore: Sized + Send + Sync + 'static {
|
|||||||
delegate.add_method("ethcore_defaultExtraData", Ethcore::default_extra_data);
|
delegate.add_method("ethcore_defaultExtraData", Ethcore::default_extra_data);
|
||||||
|
|
||||||
delegate.add_method("ethcore_vmTraceCall", Ethcore::vm_trace_call);
|
delegate.add_method("ethcore_vmTraceCall", Ethcore::vm_trace_call);
|
||||||
delegate.add_method("ethcore_diffCall", Ethcore::diff_call);
|
delegate.add_method("ethcore_stateDiffCall", Ethcore::state_diff_call);
|
||||||
|
|
||||||
delegate
|
delegate
|
||||||
}
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user