Open saona-raimundo opened 1 year ago
One option would be to just cfg out the uses of time. Would accept a patch for this.
I gave it a try, putting all uses of std::time
behind a feature...
But the use of deadline: Option<Instant>
is extensively used because
None
. So the actual implementation uses deadlines through the code.algorithms\myers\conquer
uses deadlines in the logic.Taking the code as it is and trying to include the feature superficially actually ends up duplicating quite some code.
Would you reconsider the instant
crate?
This project seems to compile it as WASM: https://github.com/xfbs/diff.rs
Related: I also would love to see a simple online text diffing tool similar to https://neil.fraser.name/software/diff_match_patch/demos/diff.html.
Hi! Thanks for the crate!
Goal
I was trying to do a web demo of
similar
. In particular, of theterminal-inline
example.Problem
The problem I faced is the use of the
std::time
module, which does not compile to Wasm.Solutions
One solution is to hide the related functionalities behind a feature.
The one I would prefer is to simply change for an implementation of
std::time
that works on Wasm, like theinstant
crate. It has bothInstance
andDuration
, which allsimilar
uses.References
instant
crate