Skip to content

Inline entity string verifier#573

Open
themuffinator wants to merge 1 commit intomainfrom
codex/update-verifyentitystring-error-handling
Open

Inline entity string verifier#573
themuffinator wants to merge 1 commit intomainfrom
codex/update-verifyentitystring-error-handling

Conversation

@themuffinator
Copy link
Owner

Summary

  • inline VerifyEntityString into g_local.hpp, keeping brace-depth tracking and safer EOF handling without a separate header
  • rely on the shared inline verifier from g_spawn.cpp and update the override truncation test to include only existing sources
  • escape the brace expectation message so format-string validation succeeds

Testing

  • g++ -std=gnu++23 -I src -I src/fmt -I src/json tests/test_entity_override_truncation.cpp -o /tmp/test_entity_override && /tmp/test_entity_override (fails: std::sinf not available in this toolchain)

Codex Task

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant