grassrootseconomics/openethereum
5
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fix todo in ethcore/types::Receipt constructor (#9086)

Browse Source 
Remove needless mutable variable and assignment
This commit is contained in:
Niklas Adolfsson 2018-07-16 13:43:14 +02:00 committed by André Silva
parent a9c93c797d
commit edb228839e
1 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
ethcore/types/src/receipt.rs
View File

@ -20,7 +20,7 @@ use ethereum_types::{H160, H256, U256, Address, Bloom};
use heapsize::HeapSizeOf; use heapsize::HeapSizeOf;
use rlp::{Rlp, RlpStream, Encodable, Decodable, DecoderError}; use rlp::{Rlp, RlpStream, Encodable, Decodable, DecoderError};
use {BlockNumber}; use BlockNumber;
use log_entry::{LogEntry, LocalizedLogEntry}; use log_entry::{LogEntry, LocalizedLogEntry};
/// Transaction outcome store in the receipt. /// Transaction outcome store in the receipt.
@ -49,12 +49,12 @@ pub struct Receipt {
impl Receipt { impl Receipt {
/// Create a new receipt. /// Create a new receipt.
pub fn new(outcome: TransactionOutcome, gas_used: U256, logs: Vec<LogEntry>) -> Receipt { pub fn new(outcome: TransactionOutcome, gas_used: U256, logs: Vec<LogEntry>) -> Self {
Receipt { Self {
gas_used: gas_used, gas_used,
log_bloom: logs.iter().fold(Bloom::default(), |mut b, l| { b = &b | &l.bloom(); b }), //TODO: use |= operator log_bloom: logs.iter().fold(Bloom::default(), |b, l| b | l.bloom()),
logs: logs, logs,
outcome: outcome, outcome,
} }
} }
} }