Skip to content

fix: Remove unused captures in example code#1604

Merged
braindigitalis merged 2 commits into
brainboxdotcc:devfrom
mikomikotaishi:fix/unused-captures
Jul 5, 2026
Merged

fix: Remove unused captures in example code#1604
braindigitalis merged 2 commits into
brainboxdotcc:devfrom
mikomikotaishi:fix/unused-captures

Conversation

@mikomikotaishi

@mikomikotaishi mikomikotaishi commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

This PR removes unused captures from lambdas, which should resolve some previous warnings during example builds.

Additionally, I have changed the alignments of the ampersand to be on the left, to be consistent with most examples (which do this already; the code examples would sometimes use three different alignments in the same file).

This continues to decrease the surface area of #1530.

Code change checklist

  • I have ensured that all methods and functions are fully documented using doxygen style comments.
  • My code follows the coding style guide.
  • I tested that my change works before raising the PR.
  • I have ensured that I did not break any existing API calls.
  • I have not built my pull request using AI, a static analysis tool or similar without any human oversight.

@netlify

netlify Bot commented Jul 1, 2026

Copy link
Copy Markdown

Deploy Preview for dpp-dev ready!

Name Link
🔨 Latest commit 8bae654
🔍 Latest deploy log https://app.netlify.com/projects/dpp-dev/deploys/6a4a6c2ee3779800089a12fa
😎 Deploy Preview https://deploy-preview-1604--dpp-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Jul 1, 2026
@braindigitalis
braindigitalis merged commit 574dbcf into brainboxdotcc:dev Jul 5, 2026
12 checks passed
@mikomikotaishi
mikomikotaishi deleted the fix/unused-captures branch July 5, 2026 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants