From 8142ece8461854ad08299de6f5b797387fd9f371 Mon Sep 17 00:00:00 2001 From: Jack Grigg Date: Fri, 29 May 2020 20:58:54 +1200 Subject: [PATCH] group: Remove "compressed" notion from GroupEncoding A generic group has a single encoding; for elliptic curves, this happens to be the compressed encoding. --- src/groth16/mod.rs | 14 +++++++------- src/groth16/tests/dummy_engine.rs | 8 ++++---- 2 files changed, 11 insertions(+), 11 deletions(-) diff --git a/src/groth16/mod.rs b/src/groth16/mod.rs index 0d85c0f..2602bc7 100644 --- a/src/groth16/mod.rs +++ b/src/groth16/mod.rs @@ -38,19 +38,19 @@ impl PartialEq for Proof { impl Proof { pub fn write(&self, mut writer: W) -> io::Result<()> { - writer.write_all(self.a.to_compressed().as_ref())?; - writer.write_all(self.b.to_compressed().as_ref())?; - writer.write_all(self.c.to_compressed().as_ref())?; + writer.write_all(self.a.to_bytes().as_ref())?; + writer.write_all(self.b.to_bytes().as_ref())?; + writer.write_all(self.c.to_bytes().as_ref())?; Ok(()) } pub fn read(mut reader: R) -> io::Result { let read_g1 = |reader: &mut R| -> io::Result { - let mut g1_repr = ::Compressed::default(); + let mut g1_repr = ::Repr::default(); reader.read_exact(g1_repr.as_mut())?; - let affine = E::G1Affine::from_compressed(&g1_repr); + let affine = E::G1Affine::from_bytes(&g1_repr); let affine = if affine.is_some().into() { Ok(affine.unwrap()) } else { @@ -70,10 +70,10 @@ impl Proof { }; let read_g2 = |reader: &mut R| -> io::Result { - let mut g2_repr = ::Compressed::default(); + let mut g2_repr = ::Repr::default(); reader.read_exact(g2_repr.as_mut())?; - let affine = E::G2Affine::from_compressed(&g2_repr); + let affine = E::G2Affine::from_bytes(&g2_repr); let affine = if affine.is_some().into() { Ok(affine.unwrap()) } else { diff --git a/src/groth16/tests/dummy_engine.rs b/src/groth16/tests/dummy_engine.rs index 5fb661b..3ddb98c 100644 --- a/src/groth16/tests/dummy_engine.rs +++ b/src/groth16/tests/dummy_engine.rs @@ -446,17 +446,17 @@ impl CurveAffine for Fr { } impl GroupEncoding for Fr { - type Compressed = FakePoint; + type Repr = FakePoint; - fn from_compressed(_bytes: &Self::Compressed) -> CtOption { + fn from_bytes(_bytes: &Self::Repr) -> CtOption { unimplemented!() } - fn from_compressed_unchecked(_bytes: &Self::Compressed) -> CtOption { + fn from_bytes_unchecked(_bytes: &Self::Repr) -> CtOption { unimplemented!() } - fn to_compressed(&self) -> Self::Compressed { + fn to_bytes(&self) -> Self::Repr { unimplemented!() } }