persistent-scratch icon indicating copy to clipboard operation
persistent-scratch copied to clipboard

Twice restoring in persistent-scratch-setup-default

Open seagle0128 opened this issue 5 years ago • 3 comments

This will cause auto-restore twice in persistent-scratch-setup-default. Or persistent-scratch-setup-default is unnecessary now?

https://github.com/Fanael/persistent-scratch/blob/fd690e138459e0b490f1fda469811a9cbcb34df7/persistent-scratch.el#L273

seagle0128 avatar Dec 31 '19 11:12 seagle0128

Nice catch, persistent-scratch-setup-default is pretty much equivalent to just enabling persistent-scratch-autosave-mode since f67327298fb0818598b5e9ef2952526b09ac2d34. Calling persistent-scratch--auto-restore twice is harmless though, any calls after the first one are no-ops.

Fanael avatar Jan 01 '20 06:01 Fanael

I'll leave this report open for now to keep myself reminded that I need to clean up this API somehow.

Fanael avatar Jan 01 '20 06:01 Fanael

Thank you for the following up!

seagle0128 avatar Jan 01 '20 09:01 seagle0128