Skip to content

Use new identifier forest API#3

Draft
BruJu wants to merge 1 commit intomasterfrom
new_identifier_forest
Draft

Use new identifier forest API#3
BruJu wants to merge 1 commit intomasterfrom
new_identifier_forest

Conversation

@BruJu
Copy link
Owner

@BruJu BruJu commented Jan 19, 2021

  • Use new identifier forest API from Identifier Forest Rewrite wasmify-sophia#10
  • Intersection / Union / Difference / Contains implementations are moved to the identifier-forest crate
  • quad_forest and lib.rs files merged for simplicity
    • lib.rs was basically empty
    • the wasm-tree-backend crate isn't supposed to do a lot of thing, except providing a thin wrapper around identifier-forest with some WebAssembly / WasmTreeFrontEnd specific small extra features

When/if this new identifier forest API will be/is stabilized and merged, this PR will be ready to be merged (all the front end unit tests when using the new front-end pass).

@BruJu BruJu marked this pull request as draft January 19, 2021 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant