Clipboard History

[ajpalkovic]

Yea, I agree, but the chances of two commands running at the same time are pretty rare. That's the only way a threading problem could have occurred. I would have to like cut and paste at the same time, that hardly every happens, there is always a delay of at least .1 or .2 seconds as my finger switches keys, idk. O well, doesn't really matter in the grand scheme of things.

[optilude]

I think no matter how likely or unlikely, you never know. You should always program for thread safety when code is executed in a multi-threaded environment. Python makes it pretty easy, thankfully.

[ajpalkovic]

Actually, now that I think about it, that is not the source of the problem.

The only way a threading issue can occur is if two commands are run at the same time. But that cannot happen in sublime. Sublime is still effectively single threaded for plugins. Jon more or less has said as much in the forums. In other words, sublime will never allow the cut and paste commands to run at the same time.

Even if my fingers magically instantaneously trigger both commands at the same time, sublime will still have a queue or something to execute one, then the other. Otherwise, EVERY SINGLE plugin would need to be updated to have a lock on their global data, which I know most don't do right now.

[colinta]

I've forked and tweaked the gist above. I would like to publish it to Package Control as ClipboardManager (to differentiate it from the existing ClipboardHistory, which doesn't do all that aptitude's plugin does).

https://gist.github.com/1590661

[ajpalkovic]

I actually stopped using the plugin a while back. I would love for it to work, but it didn't play well with yank in vi mode for instance so it became useless for me.

Sublime really needs an on_before_command and on_after_command callbacks that plugins can implement.

[and7ey]

OMG, Clipboard Manager's readme leads to this topic. I've spent 30 minutes trying to understand why clipboard_history_* doesn't work. I should use clipboard_manager_* instead...

[colinta]

I'll update the README to be a little clearer on that difference.