Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix typos #537

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,7 @@ library FiatShamirTranscript {
/// of stepping
}

/// We append a point of a N/pN x N/pN where p is less than 2^256 and the point is encoded as (32 bytes, 32 bytes)
/// We append a point of an N/pN x N/pN where p is less than 2^256 and the point is encoded as (32 bytes, 32 bytes)
/// On eth mainnet this will be a point on the bn256 pairing curve which there is a precompile for
/// WARN - This function assumes that the caller has done the mod to ensure the top bits are zero for x and y
/// @param transcript The transcript we are hashing the value into
Expand Down
2 changes: 1 addition & 1 deletion jolt-evm-verifier/src/subprotocols/HyperKZG.sol
Original file line number Diff line number Diff line change
Expand Up @@ -164,7 +164,7 @@ contract HyperKZG {
accumulated_q = q * accumulated_q;
}

// Finally we do a MSM to get the value of the the left hand side
// Finally we do an MSM to get the value of the the left hand side
// NOTE - This is gas inefficient and grows with log of the proof size so we might want
// to move to a pippenger window algo with much smaller MSMs which we might save gas on.
// Our first value is the c_x c_y as this would be the first entry of com in rust.
Expand Down