calyxir/calyx

Queues: use the nicer runner

Closed this issue · 0 comments

In #1736 I did some work that has cleaned up the "queue caller" routine. Nothing particularly deep happened there, I just got a tiny bit better at using the eDSL and thinking Calyx-ily. To make development easier, I made all my changes in a separate method (which assumes that you wish to create a non-main component that will be invoked repeatedly by another component) and left the other uses of the queue caller to use the old style (which assumes that you wish to create a main component that runs a list of commands to completion).

This is an issue to remind me to catch the other users of the queue caller up to the new style. The two methods can just be unified, with a flag or two to indicate whether the eDSL method is constructing a main-style component or a regular component.

Originally posted by @anshumanmohan in #1736 (comment)