Skip to content
This repository has been archived by the owner on Nov 15, 2017. It is now read-only.

Revisit and cleanup URI parsing code #204

Closed
gorhill opened this issue Mar 9, 2014 · 3 comments
Closed

Revisit and cleanup URI parsing code #204

gorhill opened this issue Mar 9, 2014 · 3 comments

Comments

@gorhill
Copy link
Owner

gorhill commented Mar 9, 2014

It's old and overcomplicated, and I don't like it. Clean it up, there are inefficiencies as a result of the current code.
@gorhill
Copy link
Owner Author

gorhill commented Mar 9, 2014

References: https://en.wikipedia.org/wiki/URI_scheme#Examples, http://tools.ietf.org/html/rfc3986#page-50

@gorhill
Copy link
Owner Author

gorhill commented Mar 10, 2014

Fixed in cffc6b8.

@gorhill gorhill closed this as completed Mar 10, 2014
@gorhill
Copy link
Owner Author

gorhill commented Mar 10, 2014

Relevant: http://jsperf.com/old-uritools-vs-new-uritools

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@gorhill