Skip to content
This repository has been archived by the owner on Dec 4, 2022. It is now read-only.

Implement interpretation of Read(min, max) instruction #15

Open
sergei-shabanau opened this issue Apr 15, 2020 · 2 comments
Open

Implement interpretation of Read(min, max) instruction #15

sergei-shabanau opened this issue Apr 15, 2020 · 2 comments

Comments

@sergei-shabanau
Copy link
Member

  1. I think it should be
Read(min: Int, max: Int)

in CodecVM

  1. The use case
case Read(min, max) =>

to be implemented in def interpret where min and max are not both 1.

@jdegoes
Copy link
Member

jdegoes commented Apr 15, 2020

You won't have such instruction in JVM. It may be useful to compile to just jumps (loops + conditions) and state.

@sergei-shabanau
Copy link
Member Author

True. We have defined

case class Read(min: Option[Int], max: Option[Int])  extends CodecVM

but if it becomes just

case object Read

then this issue is not needed.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants