Change pedersen hash functions in the stdlib to return a Pedersen struct

[kevaundray]

Problem

Currently users need to write code that looks like this:

let x = dep::std::hash::pedersen_with_separator([a,b,c],value)[0];

Writing [0] to get the x value hurts readability.

Happy Case

We can create a struct like so:

struct PedersenPoint {
   x : Field,
   y : Field,
}

Then change pedersen_with_separator to be named __pedersen_with_separator while the pub keyword is not in full effect yet.

Then have pedersen_with_seperator look something like:

fn pedersen_with_separator(...) -> PedersenPoint{
  let values = __pedersen_with_separator(...);
  PedersenPoint {
     x: values[0],
     y: values[1],
  }
}

The result would then be:

let x = dep::std::hash::pedersen_with_separator([a,b,c],value).x;

Alternatives Considered

No response

Additional Context

This is a breaking change

Would you like to submit a PR for this Issue?

No

Support Needs

No response