Skip to content

Conversation

@tarasmadan
Copy link
Collaborator

It was vibe coded as an experiment.

@tarasmadan tarasmadan force-pushed the context-is-ctx branch 3 times, most recently from f31fd1c to 30982a5 Compare January 21, 2026 19:42
@tarasmadan tarasmadan marked this pull request as ready for review January 21, 2026 19:48
@tarasmadan tarasmadan requested review from dvyukov and removed request for eepeep, glpesk and mvanotti January 21, 2026 19:48
@tarasmadan tarasmadan force-pushed the context-is-ctx branch 2 times, most recently from a80c473 to d051fa8 Compare January 21, 2026 20:05
@tarasmadan tarasmadan force-pushed the context-is-ctx branch 3 times, most recently from b68fa27 to 3521dcd Compare January 21, 2026 21:10
@tarasmadan tarasmadan enabled auto-merge January 21, 2026 21:10
@dvyukov
Copy link
Collaborator

dvyukov commented Jan 21, 2026

Nice!

1. It is the first parameter everywhere except tests.
2. It is the second param in the tests.
3. It is always named ctx.
4. For the cases with multiple contexts recommend to opt-out syz-linter.
@tarasmadan tarasmadan added this pull request to the merge queue Jan 21, 2026
Merged via the queue into google:master with commit a16aed1 Jan 21, 2026
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants