Skip to content

Commit

Permalink
940 - comments/typos
Browse files Browse the repository at this point in the history
  • Loading branch information
jeanmon committed Jul 27, 2023
1 parent a43e524 commit 1f34198
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -128,7 +128,7 @@ export class ClientTxExecutionContext {
// Remove notes which were already nullified during this transaction.
const dbNotesFiltered = dbNotes.filter(n => !this.pendingNullifiers.has(n.nullifier as Fr));

// Nullified pending Notes are already removed from the list.
// Nullified pending notes are already removed from the list.
const notes = pickNotes([...dbNotesFiltered, ...pendingNotes], {
sortBy: sortBy.map(field => +field),
sortOrder: sortOrder.map(field => +field),
Expand Down Expand Up @@ -217,7 +217,7 @@ export class ClientTxExecutionContext {
*/
public nullifyPendingNotes(innerNoteHash: Fr, contractAddress: AztecAddress, storageSlot: Fr) {
// IMPORTANT: We do need an in-place array mutation of this.pendingNotes as this array is shared
// by reference among the nested call. That is the main reason for 'splice' usage below.
// by reference among the nested calls. That is the main reason for 'splice' usage below.
this.pendingNotes.splice(
0,
this.pendingNotes.length,
Expand Down
2 changes: 1 addition & 1 deletion yarn-project/acir-simulator/src/client/db_oracle.ts
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ export interface NoteData {
* Information about a note needed during execution.
*/
export interface PendingNoteData extends NoteData {
/** The nullified commitment. */
/** The inner note hash (used as a nullified commitment). */
innerNoteHash: Fr;
}

Expand Down

0 comments on commit 1f34198

Please sign in to comment.