Skip to content

fix possible segfault#41

Merged
w1am merged 1 commit intomainfrom
fix-segfault
Mar 24, 2026
Merged

fix possible segfault#41
w1am merged 1 commit intomainfrom
fix-segfault

Conversation

@rb-kurrent
Copy link
Copy Markdown
Contributor

When subscribing to a stream that doesn't exist, you get a caught up message with no stream revision or position.

There is an if/else that needs to be an if/elseif to handle this possibility.

When subscribing to a stream that doesn't exist, you get a caught up
message with no stream revision or position.

There is an if/else that needs to be an if/elseif to handle this
possibility.
@rb-kurrent rb-kurrent requested a review from w1am March 23, 2026 19:56
Copy link
Copy Markdown
Contributor

@w1am w1am left a comment

Choose a reason for hiding this comment

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

LGTM

@w1am w1am merged commit ac01f51 into main Mar 24, 2026
18 checks passed
@w1am w1am deleted the fix-segfault branch March 24, 2026 10:27
w1am pushed a commit that referenced this pull request Mar 24, 2026
When subscribing to a stream that doesn't exist, you get a caught up
message with no stream revision or position.

There is an if/else that needs to be an if/elseif to handle this
possibility.
w1am pushed a commit that referenced this pull request Mar 24, 2026
When subscribing to a stream that doesn't exist, you get a caught up
message with no stream revision or position.

There is an if/else that needs to be an if/elseif to handle this
possibility.
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