search

nodejs / llparse

Generating parsers in LLVM IR
http://llparse.org
Other
584 stars 30 forks source link

streaming execute #30

Closed drom closed 4 years ago

drom commented 4 years ago

I am trying to make streaming execute to work as it was discussed in #29 .

I struggle to make it work. Since the first chunk _execute() was incomplete, it produced an error. Second _execute() did not even run for some reason?

Here is the test: https://github.com/wavedrom/vcd/blob/master/test/basic.js#L49

drom commented 4 years ago

I noticed that I can clean status->error before running execute and it helps.

indutny commented 4 years ago

I'd love to take a look. How can I reproduce this?

indutny commented 4 years ago

FWIW, tests appear to pass in your repo when I run it locally.

drom commented 4 years ago

Sorry for a false alarm, it was my bug. It fixed it somehow in the process.