search

serokell / universum

:milky_way: Prelude written in @Serokell
MIT License
176 stars 28 forks source link

Shorten 'evaluatingStateT' and family #131

Closed chshersh closed 6 years ago

chshersh commented 6 years ago

evaluatingStateT is longer than flip runStateT. Many people complained about that. So I think we can shorten this. I propose to have evaling/execing. Compare:

evaluatingStateT
flip runStateT
evalingStateT

Though, maybe it's really minor and just a micro-optimization 🤷‍♀️

volhovm commented 6 years ago

run/exec/evalSomethingT is not that common to change the lib interface imho.

chshersh commented 6 years ago

@volhovm Sorry, I don't understand your comment... Are you for shortening or not?

volhovm commented 6 years ago

Against it.

chshersh commented 6 years ago

Okay, looks like not much people care. So I'm closing this.