Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: esau <152162806+sklppy88@users.noreply.github.com>
  • Loading branch information
benesjan and sklppy88 authored Feb 22, 2024
1 parent 809ff45 commit 109b9ad
Show file tree
Hide file tree
Showing 5 changed files with 5 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ graph TD;
CurrentM --> Value1[Current Value 1]
CurrentM --> Value2[Current Value 2]
CurrentM --> ValueN[Current Value n]
Pending --> PendingM[pending note hash 1]
Pending --> PendingM[Pending Note Hash 1]
PendingM --> PValue1[Pending Value 1]
PendingM --> PValue2[Pending Value 2]
PendingM --> PValueN[Pending Value n]
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -302,7 +302,7 @@ pub fn compute_unique_siloed_note_hash(nonce: Field, siloed_note_hash: Field) ->
)
}

pub fn compute_unique_siloed_note_hashs<N>(
pub fn compute_unique_siloed_note_hashes<N>(
first_nullifier: Field,
siloed_note_hashs: [SideEffect; N]
) -> [SideEffect; N] {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -440,7 +440,7 @@ export class CrossChainTestHarness {
}

async redeemShieldPrivatelyOnL2(shieldAmount: bigint, secret: Fr) {
this.logger('Spending note hash in private call');
this.logger('Spending note in private call');
const privateTx = this.l2Token.methods.redeem_shield(this.ownerAddress, shieldAmount, secret).send();
const privateReceipt = await privateTx.wait();
expect(privateReceipt.status).toBe(TxStatus.MINED);
Expand Down
2 changes: 1 addition & 1 deletion yarn-project/pxe/src/kernel_prover/proof_creator.ts
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ export interface ProofCreator {
/**
* Computes the siloed commitments for a given set of public inputs.
*
* @param publicInputs - The public inputs containing the contract address and new note hashes to be used in generating siloed commitments.
* @param publicInputs - The public inputs containing the contract address and new note hashes to be used in generating siloed note hashes.
* @returns An array of Fr (finite field) elements representing the siloed commitments.
*/
getSiloedCommitments(publicInputs: PrivateCircuitPublicInputs): Promise<Fr[]>;
Expand Down
2 changes: 1 addition & 1 deletion yarn-project/simulator/src/public/execution.ts
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ export interface PublicExecutionResult {
execution: PublicExecution;
/** The return values of the function. */
returnValues: Fr[];
/** The new note hashes to be inserted into the commitments tree. */
/** The new note hashes to be inserted into the note hashes tree. */
newNoteHashes: SideEffect[];
/** The new l2 to l1 messages generated in this call. */
newL2ToL1Messages: L2ToL1Message[];
Expand Down

0 comments on commit 109b9ad

Please sign in to comment.