Avoid deriving CanonicalSerialize/CanonicalDeserialize for relevant structs and overriding default implementations with concrete ones to get the expected behaviour (e.g. deserialize_uncompressed calls the deserialize(compressed) if the function is not overridden, but that's not what we may want) (please review this carefully, we want to make sure that no (unexpected) default implementation is called);
Making sure that canonical_serialize_without_metadata() will act like the previous ToBytes/FromBytes (preparing for a future PR in which we are going to remove them): meaning all data is saved in uncompressed form and no info regarding length or types of data structures are saved (please review this carefully too, as we currently use the to_bytes! in Fiat Shamir sponges);
This PR aims at:
canonical_serialize_without_metadata()will act like the previous ToBytes/FromBytes (preparing for a future PR in which we are going to remove them): meaning all data is saved in uncompressed form and no info regarding length or types of data structures are saved (please review this carefully too, as we currently use the to_bytes! in Fiat Shamir sponges);