Skip to content

Fix script confirmer - #33

Merged
HealthcareICT merged 2 commits into
dvtk-org:masterfrom
bennert:fix-script-confirmer
Aug 3, 2026
Merged

HealthcareICT merged 2 commits into
dvtk-org:masterfrom
bennert:fix-script-confirmer

Conversation

@bennert

@bennert bennert commented Aug 3, 2026 •

Copy link
Copy Markdown
Collaborator

Summary
This PR contains two related fixes:

  • Fix script parser confirmer interaction handling.
  • Allow unquoted INFO(...) text to be parsed as STRING in Definition lexer conditions.

Why

  • The script confirmer flow needed more reliable parser interaction handling.
  • Some Definition files can contain condition text in the form : INFO(...); this should parse the same way as the quoted form : "INFO(...)".

What changed
Script parser confirmer handling

  • Updated script parser logic in: script_parser.y
  • Includes related generated parser updates in Source/Libraries/Scripting.

Definition lexer support for unquoted INFO(...)

  • Added lexer support to tokenize INFO(...) as STRING in:

    • definition_lex.l

    • definition_lex.l

    • definition_lex.cpp

Validation
Built the Definition parser/library successfully for the Source tree:

  • Definition.vcxproj

@bennert
bennert requested a review from HealthcareICT August 3, 2026 19:53
@HealthcareICT
HealthcareICT merged commit 298b7a2 into dvtk-org:master Aug 3, 2026
2 checks passed
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