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

Fix misc compile warnings #11258

Merged
merged 1 commit into from
Nov 13, 2019
Merged

Fix misc compile warnings #11258

merged 1 commit into from
Nov 13, 2019

Conversation

dvdplm
Copy link
Collaborator

@dvdplm dvdplm commented Nov 13, 2019

Not much to say here.

@dvdplm dvdplm self-assigned this Nov 13, 2019
@dvdplm dvdplm added A0-pleasereview 🤓 Pull request needs code review. A2-insubstantial 👶 Pull request requires no code review (e.g., a sub-repository hash update). B0-patch-stable 🕷 Pull request should also be back-ported to the stable branch. B1-patch-beta 🕷🕷 M4-core ⛓ Core client code / Rust. labels Nov 13, 2019
pub struct Bn128Pairing;

#[derive(Debug)]
/// The Blake2F builtin
Copy link
Collaborator

Choose a reason for hiding this comment

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

❤️

@niklasad1 niklasad1 added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Nov 13, 2019
@niklasad1 niklasad1 merged commit e194a2c into master Nov 13, 2019
@niklasad1 niklasad1 deleted the dp/chore/fix-compiler-warnings branch November 13, 2019 23:01
dvdplm added a commit that referenced this pull request Nov 20, 2019
* master:
  Clarify what first_block `None` means (#11269)
  removed redundant VMType enum with one variant (#11266)
  Ensure jsonrpc threading settings are sane (#11267)
  Return Ok(None) when the registrar contract returns empty slice (#11257)
  Add a benchmark for snapshot::account::to_fat_rlps() (#11185)
  Fix misc compile warnings (#11258)
  simplify verification (#11249)
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). A8-looksgood 🦄 Pull request is reviewed well. B0-patch-stable 🕷 Pull request should also be back-ported to the stable branch. M4-core ⛓ Core client code / Rust.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants