yonote/yjs
14
0
Fork 0
Code Pull Requests 1 Releases Activity

Merge pull request #439 from Synthesia-Technologies/feat/deterministic-update-encoding

Browse Source 
Make encodeStateAsUpdate deterministic
This commit is contained in:
Kevin Jahns
2023-02-21 10:59:31 +01:00
committed by GitHub
parent ea7ad07f34 6b7b3136e0
commit 58b754950e
1 changed files with 15 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/utils/DeleteSet.js
View File

@@ -219,7 +219,11 @@ export const createDeleteSetFromStructStore = ss => {
*/ */
export const writeDeleteSet = (encoder, ds) => { export const writeDeleteSet = (encoder, ds) => {
encoding.writeVarUint(encoder.restEncoder, ds.clients.size) encoding.writeVarUint(encoder.restEncoder, ds.clients.size)
ds.clients.forEach((dsitems, client) => {
// Ensure that the delete set is written in a deterministic order
Array.from(ds.clients.entries())
.sort((clientA, clientB) => clientA[0] - clientB[0])
.forEach(([client, dsitems]) => {
encoder.resetDsCurVal() encoder.resetDsCurVal()
encoding.writeVarUint(encoder.restEncoder, client) encoding.writeVarUint(encoder.restEncoder, client)
const len = dsitems.length const len = dsitems.length