From d74c0ed7ebd39ee00f0292602e898265b1de7d40 Mon Sep 17 00:00:00 2001 From: Lilleman auf Larv Date: Fri, 9 Oct 2026 11:30:15 +0200 Subject: [PATCH] Rescore: the paired ruling at d9f0fc9 against ee4b40b is better --- docs/comprehension-history.md | 21 +++++++++++++++++++++ 1 file changed, 21 insertions(+) diff --git a/docs/comprehension-history.md b/docs/comprehension-history.md index 9e99a2c..39140da 100644 --- a/docs/comprehension-history.md +++ b/docs/comprehension-history.md @@ -41,3 +41,24 @@ Inherited architect, decided by: - `pick.go:9` `pickKey`: a pick-key bug needs one file, where it needed four and the vocabulary - `render.go:155` `readField`: one named predicate, `keptInPick`, states the keep rule - `path.go:72` `compiledPath`: `headSpans` is the costliest new read in the diff + +## 2026-10-09T09:30:15Z, PR #190 at d9f0fc9, against ee4b40b + +Ruling: better + +| Seat | Vote | +|---|---| +| Mid A | better | +| Inherited architect | better | + +Mid A, decided by: + +- `pick.go:9` `pickKey`: the definitions and the code building and reading the keys share one file +- `render.go:155` `readField`: `keptInPick` holds the whole keep rule and its own nil check +- `path.go:74` `compiledPath`: `headSpans` and the bare 0 its callers pass cost a second read + +Inherited architect, decided by: + +- `pick.go:9` `pickKey`: the 3am mechanism reads in one file, in reading order +- `datatype.go:217` `columnRead`: beside its consumers, and `node.go:70` names its four readers +- `path.go:119` `compileStep`: six parameters, the one place that reads harder