Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix crash on startup when resizing before active_win #1114

Merged
merged 1 commit into from Aug 27, 2019

Conversation

@ailin-nemui
Copy link
Contributor

commented Aug 27, 2019

No description provided.

@ailin-nemui

This comment has been minimized.

Copy link
Contributor Author

commented Aug 27, 2019

reported by fling

@dequis

This comment has been minimized.

Copy link
Member

commented Aug 27, 2019

Nice, how did you figure it out? What are the repro steps?

@ailin-nemui

This comment has been minimized.

Copy link
Contributor Author

commented Aug 27, 2019

I did

while :; do pkill -WINCH irssi; done

and the start Irssi to simulate this race.

@dequis
dequis approved these changes Aug 27, 2019
Copy link
Member

left a comment

Awesome

@ailin-nemui

This comment has been minimized.

Copy link
Contributor Author

commented Aug 27, 2019

bdo#935813 @smcv

@ailin-nemui ailin-nemui merged commit 570d671 into irssi:master Aug 27, 2019

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@ailin-nemui ailin-nemui deleted the ailin-nemui:startup-crash branch Aug 27, 2019

@ailin-nemui ailin-nemui added this to the 1.2.2 milestone Aug 29, 2019

@ailin-nemui

This comment has been minimized.

Copy link
Contributor Author

commented Sep 3, 2019

actually a duplicate of #1053 oops!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.