Error: cannot uniquely infer foreach argument types

[arjunadeltoso]

I have this code

auto redis_vals = this.redisdb.send("LRANGE", "vals", 0, -1);
writeln(redis_vals); // This prints [val_a, val_b]
foreach (val; redis_vals)
  writeln(val);

it doesn't compile because of Error: cannot uniquely infer foreach argument types.

I'm following the example at http://adilbaig.github.io/Tiny-Redis/

for(fruit; fruits)
  writeln(fruit);

and that also seems wrongly suggesting to use for instead of foreach.

[adilbaig]

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

[arjunadeltoso]

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).

[adilbaig]

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.

[adilbaig]

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