From 158ce5dee2413c6834bb7c6e3986ef4e46865bcf Mon Sep 17 00:00:00 2001 From: wkh Date: Fri, 30 Oct 2026 10:03:52 +0100 Subject: [PATCH] =?UTF-8?q?Review=20abgeschlossen:=20awk-Br=C3=BCcke=20und?= =?UTF-8?q?=20Repo=20bleiben=20so?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- doc/review.md | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/doc/review.md b/doc/review.md index a2a14fe..b0e4672 100644 --- a/doc/review.md +++ b/doc/review.md @@ -83,7 +83,7 @@ Index ohne Entsprechung in der Suche offen. größter Bucket 223. Abhilfe: FNV-1a o. Ä., Primzahl-Slotzahl. 3. **Geld in `Float` — Behoben** (`Rational` in `total`/`mean`/`subtotal` und /act). Ursprünglich: ([act.rb:11](../lib/rdb/cmd/act.rb), `to_f` im /act-Teil und in `total`). Abhilfe: `BigDecimal` oder ganzzahlige Rappen. -4. **awk-Brücke — bewusst offen.** `awk` und `awk_text` teilen sich jetzt den Aufruf. Streaming +4. **awk-Brücke — bleibt so (Entscheidung 2026-10-30).** `awk` und `awk_text` teilen sich jetzt den Aufruf. Streaming und ein eigener Ausdrucksparser wären ein Umbau mit Verhaltensrisiko (awk-Syntax ist dokumentierte Schnittstelle). Ursprünglich: ([env.rb:119-138](../lib/rdb/env.rb)): ganze Tabelle als String durch `capture3`, kein Streaming; handgeschriebener Parser erkennt Strings und @@ -97,10 +97,10 @@ Index ohne Entsprechung in der Suche offen. `test_testall_runs_demo_script` (rund 300 Befehle nacheinander). Ursprünglich: Laufzeit ca. 66 s wegen eines Ruby-Prozesses pro Test. Fehlerfälle (Encoding, Tab in Wert, Locks, Editor) nicht abgedeckt; Tests direkt gegen `Rdb.run` im Prozess wären schneller. -8. **Repo — offen, Entscheidung beim Autor.** Ursprünglich: `demo/fact.pl`, `fact_lc.pl`, `rule.pl` sind ungetrackt; +8. **Repo — bleibt so (Entscheidung 2026-10-30).** Ursprünglich: `demo/fact.pl`, `fact_lc.pl`, `rule.pl` sind ungetrackt; `demo/testall.old` (1187 Zeilen) wäre als Golden-File-Test besser gepflegt. ## Empfohlene Reihenfolge Schwachstellen, Fehler und Verbesserungen 1–3 und 5–7 sind erledigt. -Offen: die awk-Brücke (4, bewusst) und das Repo-Aufräumen (8). +Die awk-Brücke (4) und das Repo (8) bleiben auf Entscheidung des Autors so. Das Review ist damit abgeschlossen.