let eredis compatible with R13B04 #4

Merged
merged 2 commits into from Aug 23, 2011

Projects

None yet

2 participants

@tomlion
Contributor
tomlion commented Aug 19, 2011

No description provided.

@knutin
Collaborator
knutin commented Aug 21, 2011

Hi,

Thanks for the patch. I think your change would improve if you would instead use re:run/2 which returns the matched position. If you also indent your code with 4 spaces, I will merge it. :)

@tomlion
Contributor
tomlion commented Aug 22, 2011

Hey, Knutin, I'm a newbie to Erlang, I have changed the code following your suggestion, thank you for your work!

@knutin knutin merged commit 264db8a into wooga:master Aug 23, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment