aconchillo / guile-redis

Redis module for Guile
GNU General Public License v3.0
29 stars 4 forks source link

redis-close leaks sockets #4

Closed rekado closed 4 years ago

rekado commented 4 years ago

This leaks sockets:

(do ((i 1 (1+ i)))
      ((> i 20))
    (let ((c (redis-connect)))
      (redis-close c)))

but this doesn't:

(do ((i 1 (1+ i)))
      ((> i 20))
    (let ((c (redis-connect)))
      (close-port ((@ (redis connection) redis-socket) c)))) 
aconchillo commented 4 years ago

Thanks! I'll take a look.

aconchillo commented 4 years ago

Should be fixed on master. Thanks again!