-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
url_detect()? #37
Comments
Ooh, painful. We'd have to do it in R for /that/ use case. We could do it in C++, if we wanted actual URLs, but those aren't actual URLs. |
somewhat painful. If we go for "best guess regex" then we can use gruber's fairly-comprehensive-but-not-too-aggressive regex for it https://gist.github.com/gruber/249502 and https://gist.github.com/gruber/8891611 |
Good idea! We could always have url_detect(x, strict = FALSE), where strict determines if it relies on a :// find (easy enough in C++, and fast) or a regex (harder and slow) |
Could we use stringi for this? |
Note to self: we can use C++11 for this, since everyone and their pet dog seems to be into C++11 these days. |
FWIW our pet cat is also into C++11.
…On Thu, Sep 14, 2017 at 3:00 PM, Oliver Keyes ***@***.***> wrote:
Note to self: we can use C++11 for this, since everyone and their pet dog
seems to be into C++11 these days.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#37 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAfHtoc5aAV4JXU3Pe0VobiFsy9MTNtQks5siXfkgaJpZM4Gkt8c>
.
|
vis a vis http://stackoverflow.com/questions/33773299/r-regex-to-extract-url-from-text
The text was updated successfully, but these errors were encountered: