Skip to content
This repository has been archived by the owner on Apr 8, 2023. It is now read-only.

redisConnect errors out when integer port is used. #16

Closed
s-u opened this issue Oct 14, 2013 · 1 comment
Closed

redisConnect errors out when integer port is used. #16

s-u opened this issue Oct 14, 2013 · 1 comment

Comments

@s-u
Copy link

s-u commented Oct 14, 2013

redisConnect refuses to use integer ports (e.g. redisConnect("localhost", 6379L) will throw an error). The culprit is

stopifnot(class(port) == "numeric")

which should read

stopifnot(is.numeric(port))

(integers are numeric, but they have the class "integer").

@bwlewis
Copy link
Owner

bwlewis commented Oct 14, 2013

Thanks, should be fixed in commit

98bdaad

along with a few similar type checking cases.

@bwlewis bwlewis closed this as completed Oct 14, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants