Commit 0edfc83
committed
doc: list both [cm_kind] match sites in narrowing reference
The Melange paragraph claimed the only mode-aware code was in
[Lib_file_deps.deps_of_entry_modules], but [need_impl_deps_of]
(inside [Module_compilation.lib_deps_for_module]) also matches on
[cm_kind] and explicitly names [Melange _] in the pattern. The
behaviour there is "Cmx reads trans-dep impl ocamldep; everything
else does not" — [Melange _] is in the same bucket as
[Ocaml (Cmi | Cmo)], not its own — but the doc oversimplified to
the point of inaccuracy.
Rewrite as a numbered list naming both [cm_kind] match sites and
clarifying that the second site's distinction is Cmx-vs-rest, not
OCaml-vs-Melange.
Replies to #14732 review comment r3304097840 (Copilot).
Signed-off-by: Robin Bate Boerop <me@robinbb.com>1 parent d5e0d6a commit 0edfc83
1 file changed
Lines changed: 11 additions & 4 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
564 | 564 | | |
565 | 565 | | |
566 | 566 | | |
567 | | - | |
568 | | - | |
569 | | - | |
570 | | - | |
| 567 | + | |
| 568 | + | |
| 569 | + | |
| 570 | + | |
| 571 | + | |
| 572 | + | |
| 573 | + | |
| 574 | + | |
| 575 | + | |
| 576 | + | |
| 577 | + | |
571 | 578 | | |
572 | 579 | | |
573 | 580 | | |
| |||
0 commit comments