Sitelet https://github.com/eddelbuettel/rcppredis/pull/3
Skip to content

add execv - #3

Merged
eddelbuettel merged 1 commit into
eddelbuettel:masterfrom
johnlaing:master
Jun 25, 2014
Merged

eddelbuettel merged 1 commit into
eddelbuettel:masterfrom
johnlaing:master

Conversation

@johnlaing

Copy link
Copy Markdown
Contributor

[ If I tag @armstrtw here, will he get notified? ]

This patch solves Whit's problem as reported here: http://lists.r-forge.r-project.org/pipermail/rcpp-devel/2014-June/007766.html

Example:

> redis$exec("SMEMBERS hats")
list()
> redis$exec("SADD hats 'big hat'") ## quoting doesn't work
[1] 2
> redis$exec("SMEMBERS hats")
[[1]]
[1] "hat'"

[[2]]
[1] "'big"

> redis$execv(c("SADD", "hats", "little hat")) ## vectorizing does work
[1] 1
> redis$exec("SMEMBERS hats")
[[1]]
[1] "little hat"

[[2]]
[1] "hat'"

[[3]]
[1] "'big"

eddelbuettel added a commit that referenced this pull request Jun 25, 2014
add execv

Thanks a lot for this!  I started to realize that Redis was doing the 'splitting by whitespace' so we did indeed need a different entry point.  So thanks for adding it!
@eddelbuettel
eddelbuettel merged commit 546d26a into eddelbuettel:master Jun 25, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants