Skip to content

fix(validation): remove VAR_INPUT {ref} warning (E042) (1.0.x) - #1893

Merged
volsa merged 2 commits into
release/1.0.xfrom
fix/prg-4587-remove-ref-warning-1.0.x
Sep 7, 2026
Merged

fix(validation): remove VAR_INPUT {ref} warning (E042) (1.0.x)#1893
volsa merged 2 commits into
release/1.0.xfrom
fix/prg-4587-remove-ref-warning-1.0.x

Conversation

@volsa

@volsa volsa commented Aug 27, 2026

Copy link
Copy Markdown
Member

Backport of #1892.

VAR_INPUT {ref} is used throughout the stdlib (e.g. the string functions) and is currently the cleanest way to pass read-only aggregate arguments without copying. Calls to these functions should not emit warnings, so E042 is removed. The error code stays registered and is free for the next diagnostic.

Refs: PRG-4587

🤖 Generated with Claude Code

VAR_INPUT {ref} is used throughout the stdlib (e.g. the string functions) and is currently the cleanest way to pass read-only aggregate arguments without copying. Calls to these functions should not emit warnings, so E042 is removed. The error code stays registered and is free for the next diagnostic.

Refs: PRG-4587

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Aug 27, 2026

Copy link
Copy Markdown

Build Artifacts

🐧 Linux

Artifact Link Size
deb-x86_64 Download 10.8 MB
schema Download 0.0 MB
stdlib Download 33.4 MB
plc-x86_64 Download 43.5 MB
deb-aarch64 Download 5.3 MB
plc-aarch64 Download 43.3 MB

From workflow run

🪟 Windows

Artifact Link Size
stdlib.lib Download 4.0 MB
stdlib.dll Download 0.1 MB
plc.exe Download 38.3 MB

From workflow run

@Angus-Bethke-Bachmann Angus-Bethke-Bachmann left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good 👍

@volsa
volsa merged commit 37662eb into release/1.0.x Sep 7, 2026
22 of 23 checks passed
@volsa
volsa deleted the fix/prg-4587-remove-ref-warning-1.0.x branch September 7, 2026 11:20
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