Skip to content

docs: C two-pass compare diagram in architecture.md - #34

Merged
longvo92 merged 1 commit into
mainfrom
docs/architecture-c-compare-diagram
Aug 17, 2026
Merged

docs: C two-pass compare diagram in architecture.md#34
longvo92 merged 1 commit into
mainfrom
docs/architecture-c-compare-diagram

Conversation

@longvo92

Copy link
Copy Markdown
Owner

Replaces the abstract data flow of one compare sequence diagram with a flowchart that follows two C files end to end.

The old diagram flattened the compare into three lines (pass 1 / pass 2 / label). The new one shows the whole path: scanner dispatch (added / deleted / error / both), the early exits, the C shadow, PASS 2, the three noise filters that each peel off only what they can prove — rename map, autogen-name, and the new reorder step — moved-block detection, PASS 1 labelling, and the single _status_of verdict.

Light node fills so it renders in GitHub's light and dark themes alike. ARXML/A2L follow the same two passes with their own shadow.

Docs only — no code change.

Note: rendered natively by GitHub's mermaid; verified syntactically here (balanced shapes, every node declared) but not pixel-rendered locally.

The 'data flow of one compare' sequence diagram compressed the whole compare
into three abstract 'pass 1 / pass 2 / label' lines. Replace it with a flowchart
that follows two C files end to end: the scanner dispatch (added/deleted/error/
both), the early exits, the C shadow, PASS 2, the three noise filters that each
peel off only what they can prove (rename map, autogen-name, reorder), moved-
block detection, PASS 1 labelling, and the single _status_of verdict. Light node
fills so it reads in GitHub's light and dark themes alike.
@longvo92
longvo92 merged commit 08392b2 into main Aug 17, 2026
6 checks passed
@longvo92
longvo92 deleted the docs/architecture-c-compare-diagram branch August 23, 2026 04:40
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