search

NathanReb / ocaml-api-watch

Libraries and tools to keep watch on you OCaml lib's API changes
ISC License
9 stars 9 forks source link

Improve diffing implementation #60

Open NathanReb opened 6 days ago

NathanReb commented 6 days ago

To diff the set of signatures we need to fold over a pair of maps representing each a version of the interface.

Our current implementation does by folding over both maps sequentially to be able to detect added and modified values in the first pass and removed values in the second pass.

This is not very efficient and folding over both is in fact possible using Map.merge which signature correspond exactly to what we want.

NathanReb commented 6 days ago

CC @Siddhi-agg

Siddhi-agg commented 5 days ago

@NathanReb Commenting to be assigned.