josephg / josephg/ShareJS

Remove JSON OT clone() function

Open
#33 0 comments 0 reactions 0 assignees View on GitHub
implement me someday
Dominant language
JavaScript
Stars
5k
Forks
454
PR merge metrics
No merged PRs in 30d

Description

The clone function in the JSON OT code isn't ok:
- Its slow
- It requires browser support for JSON
- It shouldn't be necessary, damnit

It should be removed.

To do so, we'll probably need a way to properly roll back operation application. If an op gets half-applied, then an exception is raised during processing, we should instead invert the applied op components and apply them to restore the state.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.