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

[WIP] Fix atomic use in aqueue #94

Merged
merged 1 commit into from Jan 6, 2018

Conversation

Projects
None yet
3 participants
@MaxLeiter
Contributor

MaxLeiter commented May 18, 2017

Credit to @Hummer12007 (need to change commit author)

Unsure if the segfaulting occuring is aqueue's fault or not - seems to happened when you select emails too fast.

@ddevault

This comment has been minimized.

Owner

ddevault commented May 21, 2017

Is this good to go?

@Hummer12007

This comment has been minimized.

Contributor

Hummer12007 commented May 21, 2017

I experience morr crashes with the pr. Didn't have time to look into it past few days. I propose holding it off until your refactoring merges.

@MaxLeiter

This comment has been minimized.

Contributor

MaxLeiter commented Jan 6, 2018

If your refactoring is on hold @SirCmpwn, can you test with this and give feedback/merge?

@ddevault

This comment has been minimized.

Owner

ddevault commented Jan 6, 2018

Good call.

@ddevault ddevault merged commit b4f00a5 into ddevault:master Jan 6, 2018

@ddevault

This comment has been minimized.

Owner

ddevault commented Jan 6, 2018

Thanks!

@ddevault

This comment has been minimized.

Owner

ddevault commented Jan 6, 2018

FWIW it's not entirely unlikely that I'll rewrite aerc from scratch at some point in the future. Depends on whether or not I can find a language I like to replace it with.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment