[Pushers N05]Prefix Increment Operator to save gas

[pepebndc]

Summary by CodeRabbit

• Chores
  • Applied code style and formatting optimizations. No functional changes to system behavior.

[coderabbitai]

📝 Walkthrough

Walkthrough

This pull request updates loop increment syntax from post-increment (i++) to pre-increment (++i) in two contract files and removes an unnecessary blank line in a constructor. These are stylistic adjustments with no impact on runtime behavior or control flow.

Changes

Cohort / File(s)	Summary
Loop Increment Updates src/contracts/block-hash-pusher/BaseBuffer.sol, src/contracts/block-hash-pusher/BlockHashArrayBuilder.sol	Changed post-increment (i++) to pre-increment (++i) in for-loops within _receiveHashes and _buildBlockHashArray methods respectively. No functional changes; purely stylistic/performance convention adjustment.
Whitespace Cleanup src/contracts/block-hash-pusher/zksync/ZkSyncBuffer.sol	Removed an empty line in the constructor after the _pusher assignment. No logic or behavior changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• [Pushers N02] Custom Errors in require Statements #73: Modifies the same functions (_receiveHashes in BaseBuffer.sol and _buildBlockHashArray in BlockHashArrayBuilder.sol)
• [Pushers N03]Variables Initialized With Their Default Values #70: Changes the same loop syntax patterns in BaseBuffer and BlockHashArrayBuilder files

Poem

🐰 Pre-increments leap, post-increments hop,
One little ++i makes cleanliness pop,
Whitespace lines vanish in the night,
Same logic shines, but syntax so bright! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: replacing post-increment (i++) with pre-increment (++i) operators in loop statements across multiple files to optimize gas usage.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/pushers-N05

---

No actionable comments were generated in the recent review. 🎉

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}