gun icon indicating copy to clipboard operation
gun copied to clipboard

rewrite ham mix

Open giraffekey opened this issue 5 years ago • 0 comments

The mix_ham function (I called it that as a joke, can be changed to ham_mix) isn't very Rust-like and is something I wrote just by directly translating the JavaScript logic without actually understanding it. In comparison, the lex_from_graph function is something where once I understood what it is doing I completely rewrote it in a better way.

mix_ham would be better off rewritten by someone who understands both Rust and the conflict resolution algorithm better than I currently do. Could be future me or someone else.

Also I think the real JS implementation is probably more complex than what was in the porting guide so we might need to just work on conflict resolution in general.


Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

giraffekey avatar Dec 14 '20 21:12 giraffekey