Error: cannot uniquely infer foreach argument types about tiny-redis HOT 4 CLOSED

Hi @arjunadeltoso,

Thanks for reporting this. The foreach is implemented using opApply with the following signature :

int opApply(int delegate(ulong k, Response value) dg)

The key is required. For now, you'll need to implement your foreach loops as foreach(k, v; redis_vals) to get it to run. I'll move to a range based implementation in a future release.

And yes, the doc should say foreach

from tiny-redis.

Thanks for the reply, yeah wasn't sure if

foreach(_, fruit; fruits)
  writeln(fruit)

was the correct approach or if I needed to overload opApply in response.d to take care of the values only iteration case (I've quickly tried it but with no success, just starting with D).

from tiny-redis.

That's correct. I've overloaded opApply in master to allow foreach without key declarations. I need to mull over making it into range before my next release. This one's a quick patch, but it closes the issue. I will close this ticket in my next release.

And the docs are fixed.

from tiny-redis.

https://github.com/adilbaig/Tiny-Redis/releases/tag/v2.0.2

from tiny-redis.