Page MenuHomePhabricator

Add Utility/UriParser
ClosedPublic

Authored by vharron on Jan 6 2015, 3:20 PM.

Details

Summary

I need a simple URI parser for parsing connect://hostname:port strings passed to "platform connect"

Diff Detail

Repository
rL LLVM

Event Timeline

vharron updated this revision to Diff 17842.Jan 6 2015, 3:20 PM
vharron retitled this revision from to Add Utility/UriParser.
vharron updated this object.
vharron edited the test plan for this revision. (Show Details)
vharron added reviewers: ovyalov, sivachandra.
vharron set the repository for this revision to rL LLVM.
vharron added a reviewer: clayborg.
vharron added a subscriber: Unknown Object (MLST).
clayborg accepted this revision.Jan 6 2015, 3:23 PM
clayborg edited edge metadata.

Looks fine.

This revision is now accepted and ready to land.Jan 6 2015, 3:23 PM
vharron updated this revision to Diff 17843.Jan 6 2015, 3:26 PM
vharron edited edge metadata.

Changed path portion of sscanf to 2047 characters (2049 buffer length - terminating null - leading /)

ovyalov accepted this revision.Jan 6 2015, 3:38 PM
ovyalov edited edge metadata.

LGTM

source/Utility/UriParser.cpp
36 ↗(On Diff #17843)

indentation.

This revision was automatically updated to reflect the committed changes.