Update sapling-crypto crate to use ff crate

Cargo.lock

@@ -365,6 +365,7 @@ dependencies = [
  "blake2-rfc 0.2.18 (git+https://github.com/gtank/blake2-rfc?rev=7a5b5fc99ae483a0043db7547fb79a6fa44b88a9)",
  "byteorder 1.2.2 (registry+https://github.com/rust-lang/crates.io-index)",
  "digest 0.7.2 (registry+https://github.com/rust-lang/crates.io-index)",
+ "ff 0.4.0 (registry+https://github.com/rust-lang/crates.io-index)",
  "hex-literal 0.1.1 (registry+https://github.com/rust-lang/crates.io-index)",
  "pairing 0.14.2",
  "rand 0.4.2 (registry+https://github.com/rust-lang/crates.io-index)",

sapling-crypto/Cargo.toml

@@ -14,6 +14,7 @@ features = ["expose-arith"]
 
 [dependencies]
 bellman = { path = "../bellman" }
+ff = "0.4"
 rand = "0.4"
 digest = "0.7"
 byteorder = "1"

sapling-crypto/src/circuit/boolean.rs

@@ -1,9 +1,5 @@
-use pairing::{
+use ff::{BitIterator, Field, PrimeField};
-    Engine,
+use pairing::Engine;
-    Field,
-    PrimeField,
-    BitIterator
-};
 
 use bellman::{
     ConstraintSystem,
@@ -806,8 +802,8 @@ impl From<AllocatedBit> for Boolean {
 #[cfg(test)]
 mod test {
     use bellman::{ConstraintSystem};
+    use ff::{Field, PrimeField};
     use pairing::bls12_381::{Bls12, Fr};
-    use pairing::{Field, PrimeField};
     use ::circuit::test::*;
     use super::{
         AllocatedBit,

sapling-crypto/src/circuit/ecc.rs

@@ -1,7 +1,5 @@
-use pairing::{
+use ff::Field;
-    Engine,
+use pairing::Engine;
-    Field
-};
 
 use bellman::{
     SynthesisError,
@@ -751,8 +749,8 @@ impl<E: JubjubEngine> MontgomeryPoint<E> {
 mod test {
     use bellman::{ConstraintSystem};
     use rand::{XorShiftRng, SeedableRng, Rand, Rng};
+    use ff::{BitIterator, Field, PrimeField};
     use pairing::bls12_381::{Bls12, Fr};
-    use pairing::{BitIterator, Field, PrimeField};
     use ::circuit::test::*;
     use ::jubjub::{
         montgomery,

sapling-crypto/src/circuit/lookup.rs

@@ -1,4 +1,6 @@
-use pairing::{Engine, Field};
+use ff::Field;
+use pairing::Engine;
+
 use super::*;
 use super::num::{
     AllocatedNum,

sapling-crypto/src/circuit/multieq.rs

@@ -1,8 +1,5 @@
-use pairing::{
+use ff::{Field, PrimeField};
-    Engine,
+use pairing::Engine;
-    Field,
-    PrimeField
-};
 
 use bellman::{
     SynthesisError,

sapling-crypto/src/circuit/multipack.rs

@@ -1,4 +1,5 @@
-use pairing::{Engine, Field, PrimeField};
+use ff::{Field, PrimeField};
+use pairing::Engine;
 use bellman::{ConstraintSystem, SynthesisError};
 use super::boolean::{Boolean};
 use super::num::Num;

sapling-crypto/src/circuit/num.rs

@@ -1,10 +1,5 @@
-use pairing::{
+use ff::{BitIterator, Field, PrimeField, PrimeFieldRepr};
-    Engine,
+use pairing::Engine;
-    Field,
-    PrimeField,
-    PrimeFieldRepr,
-    BitIterator
-};
 
 use bellman::{
     SynthesisError,
@@ -462,8 +457,8 @@ impl<E: Engine> Num<E> {
 mod test {
     use rand::{SeedableRng, Rand, Rng, XorShiftRng};
     use bellman::{ConstraintSystem};
+    use ff::{BitIterator, Field, PrimeField};
     use pairing::bls12_381::{Bls12, Fr};
-    use pairing::{Field, PrimeField, BitIterator};
     use ::circuit::test::*;
     use super::{AllocatedNum, Boolean};
 

sapling-crypto/src/circuit/pedersen_hash.rs

@@ -116,8 +116,8 @@ mod test {
     use super::*;
     use ::circuit::test::*;
     use ::circuit::boolean::{Boolean, AllocatedBit};
+    use ff::PrimeField;
     use pairing::bls12_381::{Bls12, Fr};
-    use pairing::PrimeField;
 
     #[test]
     fn test_pedersen_hash_constraints() {

sapling-crypto/src/circuit/sapling/mod.rs

@@ -1,8 +1,4 @@
-use pairing::{
+use ff::{Field, PrimeField, PrimeFieldRepr};
-    PrimeField,
-    PrimeFieldRepr,
-    Field,
-};
 
 use bellman::{
     SynthesisError,
@@ -602,7 +598,7 @@ impl<'a, E: JubjubEngine> Circuit<E> for Output<'a, E> {
 
 #[test]
 fn test_input_circuit_with_bls12_381() {
-    use pairing::{Field, BitIterator};
+    use ff::{BitIterator, Field};
     use pairing::bls12_381::*;
     use rand::{SeedableRng, Rng, XorShiftRng};
     use ::circuit::test::*;
@@ -734,7 +730,7 @@ fn test_input_circuit_with_bls12_381() {
 
 #[test]
 fn test_output_circuit_with_bls12_381() {
-    use pairing::{Field};
+    use ff::Field;
     use pairing::bls12_381::*;
     use rand::{SeedableRng, Rng, XorShiftRng};
     use ::circuit::test::*;

sapling-crypto/src/circuit/sprout/mod.rs

@@ -1,4 +1,5 @@
-use pairing::{Engine, Field};
+use ff::Field;
+use pairing::Engine;
 use bellman::{ConstraintSystem, SynthesisError, Circuit, LinearCombination};
 use circuit::boolean::{
     AllocatedBit,

sapling-crypto/src/circuit/test/mod.rs

@@ -1,9 +1,5 @@
-use pairing::{
+use ff::{Field, PrimeField, PrimeFieldRepr};
-    Engine,
+use pairing::Engine;
-    Field,
-    PrimeField,
-    PrimeFieldRepr
-};
 
 use bellman::{
     LinearCombination,
@@ -445,8 +441,8 @@ impl<E: Engine> ConstraintSystem<E> for TestConstraintSystem<E> {
 
 #[test]
 fn test_cs() {
+    use ff::PrimeField;
     use pairing::bls12_381::{Bls12, Fr};
-    use pairing::PrimeField;
 
     let mut cs = TestConstraintSystem::<Bls12>::new();
     assert!(cs.is_satisfied());

sapling-crypto/src/circuit/uint32.rs

@@ -1,8 +1,5 @@
-use pairing::{
+use ff::{Field, PrimeField};
-    Engine,
+use pairing::Engine;
-    Field,
-    PrimeField
-};
 
 use bellman::{
     SynthesisError,
@@ -415,8 +412,8 @@ mod test {
     use rand::{XorShiftRng, SeedableRng, Rng};
     use ::circuit::boolean::{Boolean};
     use super::{UInt32};
+    use ff::Field;
     use pairing::bls12_381::{Bls12};
-    use pairing::{Field};
     use ::circuit::test::*;
     use bellman::{ConstraintSystem};
     use circuit::multieq::MultiEq;

sapling-crypto/src/group_hash.rs

@@ -4,7 +4,7 @@ use jubjub::{
     edwards
 };
 
-use pairing::{
+use ff::{
     PrimeField
 };
 

sapling-crypto/src/jubjub/edwards.rs

@@ -1,10 +1,4 @@
-use pairing::{
+use ff::{BitIterator, Field, PrimeField, PrimeFieldRepr, SqrtField};
-    Field,
-    SqrtField,
-    PrimeField,
-    PrimeFieldRepr,
-    BitIterator
-};
 
 use super::{
     JubjubEngine,

sapling-crypto/src/jubjub/fs.rs

@@ -1,7 +1,9 @@
 use byteorder::{ByteOrder, LittleEndian};
-use pairing::{BitIterator, Field, PrimeField, SqrtField, PrimeFieldRepr, PrimeFieldDecodingError, LegendreSymbol};
+use ff::{
-use pairing::LegendreSymbol::*;
+    adc, mac_with_carry, sbb, BitIterator, Field,
-use pairing::{adc, sbb, mac_with_carry};
+    LegendreSymbol::{self, *},
+    PrimeField, PrimeFieldDecodingError, PrimeFieldRepr, SqrtField,
+};
 
 use super::ToUniform;
 

sapling-crypto/src/jubjub/mod.rs

@@ -17,12 +17,8 @@
 //! the Montgomery curve forms a group isomorphism, allowing points
 //! to be freely converted between the two forms.
 
-use pairing::{
+use ff::{Field, PrimeField, SqrtField};
-    Engine,
+use pairing::Engine;
-    Field,
-    PrimeField,
-    SqrtField
-};
 
 use group_hash::group_hash;
 

sapling-crypto/src/jubjub/montgomery.rs

@@ -1,10 +1,4 @@
-use pairing::{
+use ff::{BitIterator, Field, PrimeField, PrimeFieldRepr, SqrtField};
-    Field,
-    SqrtField,
-    PrimeField,
-    PrimeFieldRepr,
-    BitIterator
-};
 
 use super::{
     JubjubEngine,

sapling-crypto/src/jubjub/tests.rs

@@ -6,7 +6,7 @@ use super::{
     edwards
 };
 
-use pairing::{
+use ff::{
     Field,
     PrimeField,
     PrimeFieldRepr,

sapling-crypto/src/lib.rs

@@ -2,6 +2,7 @@ extern crate pairing;
 extern crate bellman;
 extern crate blake2_rfc;
 extern crate digest;
+extern crate ff;
 extern crate rand;
 extern crate byteorder;
 

sapling-crypto/src/pedersen_hash.rs

@@ -1,5 +1,5 @@
+use ff::{Field, PrimeField, PrimeFieldRepr};
 use jubjub::*;
-use pairing::*;
 
 #[derive(Copy, Clone)]
 pub enum Personalization {

sapling-crypto/src/primitives/mod.rs

@@ -1,8 +1,4 @@
-use pairing::{
+use ff::{Field, PrimeField, PrimeFieldRepr};
-    Field,
-    PrimeField,
-    PrimeFieldRepr
-};
 
 use constants;
 

sapling-crypto/src/redjubjub.rs

@@ -1,7 +1,7 @@
 //! Implementation of RedJubjub, a specialization of RedDSA to the Jubjub curve.
 //! See section 5.4.6 of the Sapling protocol specification.
 
-use pairing::{Field, PrimeField, PrimeFieldRepr};
+use ff::{Field, PrimeField, PrimeFieldRepr};
 use rand::{Rng, Rand};
 use std::io::{self, Read, Write};