Skip to content

Comments

Use a disposable object to encapsulate the logic of signaling the latch when an exception is thrown in the processing path#1640

Merged
mauroservienti merged 2 commits intoimprovement-optimize-peaksfrom
using-signaler
Dec 11, 2025
Merged

Use a disposable object to encapsulate the logic of signaling the latch when an exception is thrown in the processing path#1640
mauroservienti merged 2 commits intoimprovement-optimize-peaksfrom
using-signaler

Conversation

@SzymonPobiega
Copy link
Member

@SzymonPobiega SzymonPobiega commented Dec 10, 2025

Use a disposable object to encapsulate the logic of signaling the latch when an exception is thrown in the processing path

@SzymonPobiega SzymonPobiega self-assigned this Dec 10, 2025
@SzymonPobiega SzymonPobiega changed the base branch from master to improvement-optimize-peaks December 10, 2025 08:32
Co-Authored-By: Ramon Smits <ramon.smits@gmail.com>
Co-Authored-By: Tomasz Masternak <1092707+tmasternak@users.noreply.github.com>
Co-Authored-By: Szymon Pobiega <szymon@pobiega.com>
@mauroservienti mauroservienti changed the title Using signaler Use a disposable signaler Dec 10, 2025
@mauroservienti mauroservienti changed the title Use a disposable signaler Use a disposable object to encapsulate the logic of signaling the latch when an exception is thrown in the processing path Dec 10, 2025
@mauroservienti
Copy link
Member

@tmasternak @ramonsmits @SzymonPobiega, are we good with merging this one?

@mauroservienti
Copy link
Member

@xwipeoutx, what do you think about this additional improvement?

Copy link
Contributor

@xwipeoutx xwipeoutx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@mauroservienti mauroservienti merged commit ba0f8bd into improvement-optimize-peaks Dec 11, 2025
5 of 6 checks passed
@mauroservienti mauroservienti deleted the using-signaler branch December 11, 2025 05:21
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.

3 participants