search

taoensso / carmine

Redis client + message queue for Clojure
https://www.taoensso.com/carmine
Eclipse Public License 1.0
1.16k stars 131 forks source link

(wcar* (car/set "test" "")) to throw exception? #84

Closed bzg closed 10 years ago

bzg commented 10 years ago

Hi Peter,

since 2.5, (wcar* (car/set "test" "")) will throw an ArrayIndexOutOfBoundsException because of the empty string in the (car/set ...) expression.

Is it by design? My use-case is very simple: I use (car/hset ...) to set redis values from a web form, and some of these values are empty string (e.g. for hidden input in the form.)

Thanks in advance for your reply, and thanks for this great library!

ptaoussanis commented 10 years ago

Hi Bastien,

Is it by design?

Definitely not, you've spotted a bug (thanks for the report!). Have just pushed v2.6.2 with the fix.

Thanks in advance for your reply, and thanks for this great library!

You're very welcome, cheers! :-)

bzg commented 10 years ago

I confirm the fix works fine. Thanks for the prompt reply!

ptaoussanis commented 10 years ago

You're welcome, appreciate the confirmation. Cheers!