Skip to content

Commit

Permalink
add a clarifying comment
Browse files Browse the repository at this point in the history
  • Loading branch information
0age committed Feb 28, 2024
1 parent 78e7973 commit a354bde
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion src/core/lib/OrderCombiner.sol
Original file line number Diff line number Diff line change
Expand Up @@ -453,9 +453,12 @@ contract OrderCombiner is OrderFulfiller, FulfillmentApplier {
// Apply criteria resolvers to each order as applicable.
_applyCriteriaResolvers(advancedOrders, criteriaResolvers);

// Emit an event for each order signifying that it has been fulfilled.
// Iterate over each order to check authorization status (for restricted
// orders), generate orders (for contract orders), and emit events (for
// all available orders) signifying that they have been fulfilled.
// Skip overflow checks as all for loops are indexed starting at zero.
unchecked {
// Declare stack variable outside of the loop to track order hash.
bytes32 orderHash;

// Iterate over each order.
Expand Down

0 comments on commit a354bde

Please sign in to comment.