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

ethjson: impl Copy for hash type wrapper #11423

Merged
merged 1 commit into from
Feb 3, 2020

Conversation

vorot93
Copy link

@vorot93 vorot93 commented Jan 30, 2020

Hash types implement Copy themselves so I found different semantics a bit surprising here.

@parity-cla-bot
Copy link

It looks like @vorot93 signed our Contributor License Agreement. 👍

Many thanks,

Parity Technologies CLA Bot

@niklasad1
Copy link
Collaborator

niklasad1 commented Jan 31, 2020

Hash types implement Copy themselves so I found different semantics a bit surprising here.

It is because initially ethereum_types::{H64, .. H256} was not Copy so it might be more places where we are inconsistent.

@niklasad1 niklasad1 added A2-insubstantial 👶 Pull request requires no code review (e.g., a sub-repository hash update). M2-config 📂 Chain specifications and node configurations. M4-core ⛓ Core client code / Rust. labels Jan 31, 2020
@ordian ordian merged commit 3c3c19d into openethereum:master Feb 3, 2020
@vorot93 vorot93 deleted the ethjson-hash-copy branch February 3, 2020 16:04
dvdplm added a commit that referenced this pull request Feb 4, 2020
* master:
  update kvdb-rocksdb to 0.4 (#11442)
  Rough architecutre diagram. (#11396)
  ethjson: impl Copy for hash type wrapper (#11423)
  Remove dead bootnodes, add new geth bootnodes (#11441)
  goerli: replace foundation bootnode (#11433)
dvdplm added a commit that referenced this pull request Feb 4, 2020
…pstream

* master:
  Avoid long state queries when serving GetNodeData requests (#11444)
  Cargo.lock: cargo update -p kvdb-rocksdb (#11447)
  rlp_derive: cleanup (#11446)
  chore: remove unused dependencies (#11432)
  update kvdb-rocksdb to 0.4 (#11442)
  Rough architecutre diagram. (#11396)
  ethjson: impl Copy for hash type wrapper (#11423)
  Remove dead bootnodes, add new geth bootnodes (#11441)
  goerli: replace foundation bootnode (#11433)
  Update publish-docker.sh (#11428)
  Revert "[Trace] Distinguish between `create` and `create2` (#11311)" (#11427)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A2-insubstantial 👶 Pull request requires no code review (e.g., a sub-repository hash update). M2-config 📂 Chain specifications and node configurations. M4-core ⛓ Core client code / Rust.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants