Use cats-effect bracket

Open pepegar opened this issue 8 years ago • 5 comments

We should use it for interpreters, where the class instantiating InterpTrans uses the lower level interpreter as a resource that's aquired, used and released afterwards.

Apr 06 '18 13:04 pepegar

Happy to help with this one.

Jul 20 '18 10:07 miciek

Awesome @miciek ! Do you have a proposal for it? or do you prefer discussing the design a bit here?

Jul 20 '18 12:07 pepegar

I will try to propose something and then we can discuss further. Thanks :)

Jul 22 '18 19:07 miciek

Sounds good! Thanks @miciek !

Jul 22 '18 22:07 pepegar

@pepegar I gave it some time and now I wonder whether you want to implement something in each Interpreter for acquiring clients (around doReq) or maybe in InterpTrans.trans or HammockFSyntax.exec?

I know that each of them solves a different problem, so it would be nice if you shared some thoughts on this topic :) Thanks!

Aug 12 '18 11:08 miciek