search

JBenda / inkcpp

Inkle Ink C++ Runtime with JSON>Binary Compiler
MIT License
70 stars 13 forks source link

crime scene example story throws "Operation for value not supported!" #81

Closed Myonmu closed 5 months ago

Myonmu commented 6 months ago

I was just testing with the crime scene story that comes with inky. (with the command line executable) Unfortunately the first advance line call directly throws Operation for value not supported! The evaluation stack is at the Compare the prints on the knife and the window line of the story when the exception is thrown.

Looks like a not supported ink syntax?

JBenda commented 5 months ago

Thanks for bringing this to my notice. It is indeed a feature inkcpp currently is not supporting. Casting types List to bool implicit. This is an oversight, I will see that I will fix it soon.

My current guess is that an empty list will evaluate to false, I'm currently unable to find documentation for that case .

Myonmu commented 5 months ago

Thanks for bringing this to my notice. It is indeed a feature inkcpp currently is not supporting. Casting types List to bool implicit. This is an oversight, I will see that I will fix it soon.

My current guess is that an empty list will evaluate to false, I'm currently unable to find documentation for that case .

I found this: https://github.com/inkle/ink/blob/master/Documentation/WritingWithInk.md#testing-for-emptiness

JBenda commented 5 months ago

Added the missing functionality and a bit more, is currently online at fix/ListBoolOperations. The matching release will be there soon (maybe a week?)