address PR feedback

AztecProtocol · Feb 29, 2024 · cd1d7e9 · cd1d7e9
1 parent d382ddc
commit cd1d7e9
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 13 deletions.
diff --git a/yarn-project/circuits.js/src/structs/kernel/combined_accumulated_data.ts b/yarn-project/circuits.js/src/structs/kernel/combined_accumulated_data.ts
@@ -34,8 +34,6 @@ import { ReadRequestContext } from '../read_request.js';
 import { SideEffect, SideEffectLinkedToNoteHash } from '../side_effects.js';
 import { NewContractData } from './new_contract_data.js';
 
-// const log = createDebugLogger('aztec:combined_accumulated_data');
-
 /**
  * Read operations from the public state tree.
  */
@@ -340,11 +338,6 @@ export class CombinedAccumulatedData {
       ...nonRevertible.publicDataUpdateRequests,
     ].filter(x => !x.isEmpty());
 
-    // log.debug('nonSquashedWrites');
-    // for (const write of nonSquashedWrites) {
-    //   log.debug(write.toFriendlyJSON());
-    // }
-
     const squashedWrites = Array.from(
       nonSquashedWrites
         .reduce<Map<string, PublicDataUpdateRequest>>((acc, curr) => {
@@ -354,11 +347,6 @@ export class CombinedAccumulatedData {
         .values(),
     );
 
-    // log.debug('squashedWrites');
-    // for (const write of squashedWrites) {
-    //   log.debug(write.toFriendlyJSON());
-    // }
-
     const publicDataUpdateRequests = padArrayEnd(
       squashedWrites,
       PublicDataUpdateRequest.empty(),

diff --git a/yarn-project/sequencer-client/src/sequencer/public_processor.test.ts b/yarn-project/sequencer-client/src/sequencer/public_processor.test.ts
@@ -359,7 +359,8 @@ describe('public_processor', () => {
         proof,
         TxL2Logs.random(2, 3),
         TxL2Logs.random(3, 2),
-        callRequests.slice().reverse(), // unsure why we need to reverse here 😬
+        // reverse because `enqueuedPublicFunctions` expects the last element to be the front of the queue
+        callRequests.slice().reverse(),
         [ExtendedContractData.random()],
       );