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

Deduplicating crate dependencies (part 2 of n, slab)#11613

Merged
ordian merged 1 commit into
openethereum:masterfrom
shergin:pr-11468-2
Apr 7, 2020
Merged

Deduplicating crate dependencies (part 2 of n, slab)#11613
ordian merged 1 commit into
openethereum:masterfrom
shergin:pr-11468-2

Conversation

@shergin
Copy link
Copy Markdown

@shergin shergin commented Apr 7, 2020

The change includes only slab module.

#11468

@shergin
Copy link
Copy Markdown
Author

shergin commented Apr 7, 2020

cc @vorot93, @ordian, @niklasad1

Copy link
Copy Markdown
Member

@ordian ordian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! Just a small nit.

Comment thread util/network-devp2p/src/host.rs
@ordian ordian added A8-looksgood 🦄 Pull request is reviewed well. M5-dependencies 🖇 Dependencies. labels Apr 7, 2020
Copy link
Copy Markdown
Collaborator

@dvdplm dvdplm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm modulo my question.

Comment thread util/network-devp2p/src/host.rs Outdated
};

type Slab<T> = ::slab::Slab<T, usize>;
type Slab<T> = ::slab::Slab<T>;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dq: Why do we need this type alias?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, we don't need it!
I fixed it.

The change includes only `slab` module.
@ordian ordian merged commit 12cbe93 into openethereum:master Apr 7, 2020
shergin added a commit to shergin/openethereum that referenced this pull request Apr 8, 2020
…timer`, and co) (openethereum#11613)

The change includes:
```
tiny-keccak
tokio-timer
textwrap
winapi
rand
rand_xorshift
scopeguard
url
```
ordian added a commit that referenced this pull request Apr 9, 2020
* Deduplicating crate dependencies (part 3 of n, `tiny_keccak`, `tokio-timer`, and co) (#11613)

The change includes:
```
tiny-keccak
tokio-timer
textwrap
winapi
rand
rand_xorshift
scopeguard
url
```

* Update rpc/src/v1/helpers/secretstore.rs

Co-Authored-By: Andronik Ordian <write@reusable.software>

* Update util/network-devp2p/src/connection.rs

Co-Authored-By: Andronik Ordian <write@reusable.software>

Co-authored-by: Andronik Ordian <write@reusable.software>
dvdplm added a commit that referenced this pull request Apr 14, 2020
* master: (25 commits)
  Update .gitmodules (#11628)
  ethcore/res: activate ecip-1088 phoenix on classic (#11598)
  Upgrade parity-common deps to latest (#11620)
  Fix Goerli syncing (#11604)
  deps: switch to upstream ctrlc (#11617)
  Deduplicating crate dependencies (part 3 of n) (#11614)
  Deduplicating crate dependencies (part 2 of n, `slab`) (#11613)
  Actually save ENR on creation and modification (#11602)
  Activate POSDAO on xDai chain and update bootnodes (#11610)
  Activate on-chain randomness in POA Core (#11609)
  Deduplicating crate dependencies (part 1 of n) (#11606)
  Update enodes for POA Sokol (#11611)
  Remove .git folder from dogerignore file so vergen library can get build date and commit hash in the binary generatio vergen library can get build date and commit hash in the binary generation (#11608)
  Reduced gas cost for static calls made to precompiles EIP2046/1352 (#11583)
  [easy] `ethcore-bloom-journal` was renamed to `accounts-bloom` (#11605)
  Use serde_json to export hardcoded sync (#11601)
  Node Discovery v4 ENR Extension (EIP-868) (#11540)
  Fix compile warnings (#11595)
  Update version to 3.0.0-alpha.1 (#11592)
  ethcore/res: bump canon fork hash for mordor and kotti testnets (#11584)
  ...
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

A8-looksgood 🦄 Pull request is reviewed well. M5-dependencies 🖇 Dependencies.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants