Skip to content

Commit

Permalink
Merge pull request #748 from akoshelev/clean-up-no-prss
Browse files Browse the repository at this point in the history
Remove no-prss feature and implementation
  • Loading branch information
akoshelev authored Jul 17, 2023
2 parents 4c224c3 + e8cc15f commit d37a1df
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 99 deletions.
2 changes: 0 additions & 2 deletions Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,6 @@ edition = "2021"
default = ["web-app", "in-memory-infra", "tracing/max_level_trace", "tracing/release_max_level_info", "descriptive-gate"]
cli = ["comfy-table", "clap"]
enable-serde = ["serde", "serde_json"]
protocol-perf-testing = ["no-prss"]
no-prss = []
disable-metrics = []
# TODO move web-app to a separate crate. It adds a lot of build time to people who mostly write protocols
# TODO Consider moving out benches as well
Expand Down
11 changes: 2 additions & 9 deletions src/protocol/prss/mod.rs
Original file line number Diff line number Diff line change
@@ -1,13 +1,4 @@
#[cfg(not(feature = "no-prss"))]
mod crypto;
#[cfg(feature = "no-prss")]
mod no_op;

#[cfg(not(feature = "no-prss"))]
pub use crypto::{Generator, GeneratorFactory, KeyExchange, SharedRandomness};
#[cfg(feature = "no-prss")]
pub use no_op::{Generator, GeneratorFactory, KeyExchange, SharedRandomness};

use super::step::Gate;
use crate::{
rand::{CryptoRng, RngCore},
Expand All @@ -19,6 +10,8 @@ use x25519_dalek::PublicKey;
#[cfg(debug_assertions)]
use std::{collections::HashSet, fmt::Formatter};

pub use crypto::{Generator, GeneratorFactory, KeyExchange, SharedRandomness};

/// Keeps track of all indices used to generate shared randomness inside `IndexedSharedRandomness`.
/// Any two indices provided to `IndexesSharedRandomness::generate_values` must be unique.
/// As PRSS instance is unique per step, this only constrains randomness generated within
Expand Down
88 changes: 0 additions & 88 deletions src/protocol/prss/no_op.rs

This file was deleted.

0 comments on commit d37a1df

Please sign in to comment.