search

pusher / atom-pair

An Atom package that allows for epic pair programming
MIT License
1.45k stars 28 forks source link

Buffers out of sync. Possible way to reproduce. #95

Closed davibe closed 6 years ago

davibe commented 8 years ago

I tried atom-pair with @crisbit. We were in the same office. We shared one file. I started adding new lines (~20) at the top of the file then stared deleting them. While i was deleting the newlines my colleague did some UNDO (cmd+z). We ended up with 2 different buffers on the screen. This breaks consistency, badly. It should not be possible.

jpatel531 commented 8 years ago

Hi there. I've experienced this too. I'm working on a branch that detects inconsistency across clients. Next step is to call a resync. Last couple of weeks I'd been shy about releasing it as performance had been seriously affected, but that no longer seems an issue. Thanks for the steps!! I'll try to get a release out asap On 12 Nov 2015 09:03, "Davide Bertola" notifications@github.com wrote:

I tried atom-pair with @crisbit https://github.com/crisbit. We were in the same office. We shared one file. I started adding new lines (~20) at the top of the file then stared deleting them. While i was deleting the newlines my colleague did some UNDO (cmd+z). We ended up with 2 different buffers on the screen. This breaks consistency, badly. It should not be possible.

— Reply to this email directly or view it on GitHub https://github.com/pusher/atom-pair/issues/95.