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 #1300] Remove kill-ring from save-hist to avoid freezing Emacs on OS X #1843

Closed
wants to merge 1 commit into from

Conversation

tuhdo
Copy link
Contributor

@tuhdo tuhdo commented Jun 4, 2015

Since it is reported to have abnormal size on OS X and can freeze Emacs
without a way to recover: #1300

Since it is reported to have abnormal size on OS X and can freeze Emacs
without a way to recover: syl20bnr#1300
@tuhdo tuhdo changed the title [FIX] Remove kill-ring from save-hist [FIX #1300] Remove kill-ring from save-hist to avoid freezing Emacs on OS X Jun 4, 2015
@trishume
Copy link
Contributor

trishume commented Jun 4, 2015

Where does the info that this is the kill ring come from? I see no mention of the kill ring in the linked issue.

@tuhdo
Copy link
Contributor Author

tuhdo commented Jun 4, 2015

It is confirmed in this issue: #1369 (comment). @syl20bnr is ok to remove it. The OP traced down to abnormal kill ring: #1369 (comment).

@syl20bnr
Copy link
Owner

syl20bnr commented Jun 5, 2015

Thank you !
I think it is a saner default to remove it but I'll re-enable it in my config because it is pretty handy and I never ran into such issue with it.

Cherry-picked into develop branch, you can safely delete your branch.

@syl20bnr syl20bnr closed this Jun 5, 2015
@tuhdo tuhdo deleted the savehist-fix branch June 5, 2015 04:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants