Re: algo.monad state-m fetch-val bug and efficiency issue

2012-09-10 Thread Phlex
On 10/09/2012 08:59, Konrad Hinsen wrote: The maintainer is me, and I am indeed not in IRC. Thanks for your suggestions, which I have applied to the source code repository. Konrad. Thanks you. Sacha -- You received this message because you are subscribed to the Google Groups "Clojure" gro

algo.monad state-m fetch-val bug and efficiency issue

2012-09-09 Thread Konrad Hinsen
Phlex writes: > I was unable to contact a maintainer of this library on irc (though i > didn't try very hard), So here is a gist with problem statement and > solutions for both issues. The maintainer is me, and I am indeed not in IRC. Thanks for your suggestions, which I have applied to the

Re: algo.monad state-m fetch-val bug and efficiency issue

2012-09-08 Thread Stephen Compall
On Sun, 2012-09-09 at 03:33 +0200, Phlex wrote: > the docstring explicitly says that the state is a map and that the > function returns the value corresponding to this key. This does break > for a string or integer key. There is no way around it, this is a bug. Sure, but a documentation bug, or a

Re: algo.monad state-m fetch-val bug and efficiency issue

2012-09-08 Thread Phlex
You sir are nitpicking on me ! On 9/09/2012 02:05, Stephen Compall wrote: https://gist.github.com/3667614 -(key s))) ;; only works for keyword keys +(key s))) ;; works for arbitrary functions You of course are right, the key parameter could be a function and this might be useful in so

Re: algo.monad state-m fetch-val bug and efficiency issue

2012-09-08 Thread Stephen Compall
On Sat, 2012-09-08 at 07:39 +0200, Phlex wrote: > I was unable to contact a maintainer of this library on irc I suggest using http://dev.clojure.org/jira/browse/ALGOM to report algo.monads bugs. > https://gist.github.com/3667614 -(key s))) ;; only works for keyword keys +(key s))) ;; wor

algo.monad state-m fetch-val bug and efficiency issue

2012-09-07 Thread Phlex
I've been using the state monad in algo.monad, and found what i think is a bug in the fetch-val function. There is also an efficiency issue with this function. I was unable to contact a maintainer of this library on irc (though i didn't try very hard), So here is a gist with problem statement