Suspicious regexp #61

Closed
p opened this Issue Mar 3, 2012 · 1 comment

Comments

Projects
None yet
2 participants

p commented Mar 3, 2012

https://github.com/luislavena/sqlite3-ruby/blob/master/lib/sqlite3.rb

Should perhaps be RUBY_VERSION =~ /(\d+\.\d+)/, escaping ., instead?

Owner

luislavena commented Mar 3, 2012

Thank you.

Actually that is going away in favor of using
RbConfig::CONFIG["ruby_version"]

Will update code when reach my computer.

Sorry for top posting. Sent from mobile.
On Mar 3, 2012 9:15 AM, "Oleg Pudeyev" <
reply@reply.github.com>
wrote:

https://github.com/luislavena/sqlite3-ruby/blob/master/lib/sqlite3.rb

Should perhaps be RUBY_VERSION =~ /(\d+\.\d+)/, escaping ., instead?


Reply to this email directly or view it on GitHub:
luislavena#61

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment