neophenix / Redis-hiredis

Perl wrapper around hiredis client
BSD 3-Clause "New" or "Revised" License
11 stars 8 forks source link

Add redis scan command support #9

Closed pru-mike closed 4 years ago

pru-mike commented 4 years ago

Hello!

I have some old project that use this lib. I found it doesn't work properly with scan command. I fix it, and test it with redis 2.8 (it's pretty old too), test works fine. I appreciate if you review my fix.

neophenix commented 4 years ago

Thanks for the PR. I don't really have a setup to properly test any more than it looks like you already have, and I don't see anything that stands out to me to look into deeper so merged!