Skip to content

backend_v: build on current V master - #138

Merged
quaesitor-scientiam merged 2 commits into
masterfrom
fix-backend-v-new-v
Oct 26, 2026
Merged

quaesitor-scientiam merged 2 commits into
masterfrom
fix-backend-v-new-v

Conversation

@quaesitor-scientiam

Copy link
Copy Markdown
Owner

Current V master rejects n.value in map[u32]bool in backend_v.v:136, so the scheduled graph rebuild fails with building graphify.exe failed.

The fix: anon struct lookups use flat.source_type_id(...), and node text is read through value_text() because V moved literal and cast type text into source_value.

Validation (V 0.5.2 6cce632 and 080c153):

  • v -o ... cmd/cli exit 0
  • v -o ... cmd/mcp exit 0
  • v test . 2 passed / 2

Review: full read of backend_v.v (3,442 lines); detectors run. Lenient-parse flag at line 2426 is a false positive (digit guard above it); unbounded-accumulator and unsafe-block flags are pre-existing, not in this diff.

🧙 Built with WOZCODE

PythonWillRule and others added 2 commits October 26, 2026 13:32
Newer V (e.g. 6cce632) rejects `n.value in map[u32]bool` and moved
struct literal and cast type text out of `value` into `source_value`.
Look up anon struct types by source_type_id and read node text through
value_text() so graphify builds and extracts on current V master.

Co-Authored-By: WOZCODE <contact@withwoz.com>
vlang/v master (080c153) builds graphify and passes its tests with the
flat source_value API, and vlang/vc now has a snapshot for it (0a3c639).
The pins were set while V's newest commit broke the build; per the
comment above, empty them once a later commit builds.

Co-Authored-By: WOZCODE <contact@withwoz.com>
@quaesitor-scientiam
quaesitor-scientiam merged commit fffaa6b into master Oct 26, 2026
14 checks passed
@quaesitor-scientiam
quaesitor-scientiam deleted the fix-backend-v-new-v branch October 26, 2026 18:43
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.

2 participants