Refactoring URI class to comply with LLVM standards
Created by: rerpha
Description of work
Replace most of the functionality in URI with a regular expression which will find everything we need.
Removing scheme
as it isn't used anywhere.
Will mirror this on the forwarder when it is ready
Issue
DM-1192
Acceptance Criteria
Only change is removal of is_alpha()
function and replacement with std::string::find_first_of()
- same functionality just a single call
Unit Tests
Tests added for double colon in scheme and port
Tests removed for scheme (scheme has now been removed)
Other
No documentation modified.
Code Review (To be filled in by the reviewer only)
-
Is the code of an acceptable quality? -
Do the changes function as described and is it robust?
Nominate for Group Code Review (Anyone can nominate it)
Indicate if you think the code should be reviewed in a Thursday code review session.
-
Recommend for group code review
Also, nominate it on the code_review Slack channel (does someone want to automate this?).