Skip to content

fix: add spaces to the "command expected at line"'s#24

Open
xirzo wants to merge 1 commit intoOvum-Programming-Language:masterfrom
xirzo:fix/line-error-space
Open

fix: add spaces to the "command expected at line"'s#24
xirzo wants to merge 1 commit intoOvum-Programming-Language:masterfrom
xirzo:fix/line-error-space

Conversation

@xirzo
Copy link

@xirzo xirzo commented Jan 18, 2026

"Command expected at line" do have spaces after them, while other errors do have those:

Source

if (!handled) {
  TokenPtr token = session->Current();
  std::string message = "Unknown top-level declaration: " + token->GetLexeme() + " at line " +
                        std::to_string(token->GetPosition().GetLine()) + " column " +
                        std::to_string(token->GetPosition().GetColumn());

  return std::unexpected(BytecodeParserError(message));
}

That results in text concatenation:

➜  TestOvum ovumc -m main.ovum -o main.oil && ovum-vm -f main.oil
Parser error: Command expected at line11

Fixed:

➜  TestOvum ovumc -m main.ovum -o main.oil && ovum-vm -f main.oil
Parser error: Command expected at line 11

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