panic on `CREATE TABLE L(x)L` about lemon-rs HOT 4 CLOSED

alright I found it, in parse.y, this bit:

table_option(A) ::= nm(X). {
  if "strict".eq_ignore_ascii_case(&X.0) {
    A = TableOptions::STRICT;
  }else{
    A = TableOptions::NONE;
    let msg = format!("unknown table option: {}", &X);
    self.ctx.sqlite3_error_msg(&msg);
  }
}

generates:

//line 161 "src/parser/parse.y"
{
  if "strict".eq_ignore_ascii_case(&self.yy_move(0).yy222().0) {
    yylhsminor = YYMINORTYPE::yy201( TableOptions::STRICT);
  }else{
    yylhsminor = YYMINORTYPE::yy201( TableOptions::NONE);
    let msg = format!("unknown table option: {}", &self.yy_move(0).yy222());
    self.ctx.sqlite3_error_msg(&msg);
  }
}

but the call to yy222 consumes self. on the subsequent call to create the error message, the content of the name was already taken from self.minor, and it blows up.

changing the rule to:

table_option(A) ::= nm(X). {
  let name = X;
  if "strict".eq_ignore_ascii_case(&name.0) {
    A = TableOptions::STRICT;
  }else{
    A = TableOptions::NONE;
    let msg = format!("unknown table option: {name}");
    self.ctx.sqlite3_error_msg(&msg);
  }
}

it doesn't panic anymore, but doesn't return an error either. Still investigating.

from lemon-rs.

Indeed, in one action, we are supposed to consume entry once.

from lemon-rs.

I fixed it by immediately returning a ParseError after self.ctx.sqlite3_error_msg(&msg);, now it returns an error as expected. LMK if you are interested in these fixes!

from lemon-rs.

I've reapplied your fix here: a7cd0c1
And I am going to fix all usages of sqlite3_error_msg.
Thanks for all your feedback.

from lemon-rs.