[Merge] lp:~pete-woods/net-cpp/uri-builder into lp:net-cpp

Thomas Voß thomas.voss at canonical.com
Tue Jun 10 13:12:16 UTC 2014


Review: Needs Fixing

Thanks for getting started on this. I have a few comments:

(1.) I would propose to have a

struct Uri
{
struct Endpoint
{
};

struct Parameters
{
};
};

Where Endpoint and Parameters could just be appropriate typedefs. We could put it into core/net/uri.h, being ready for consumption by core/net/http/client.h.

(2.) We should provide a default implementation of build_uri, that can be reused by any implementation that provides implementations for url_escape.

(3.) There are some manual uri setups in the test-cases. I would appreciate it if you could change those to leverage the new interface.
-- 
https://code.launchpad.net/~pete-woods/net-cpp/uri-builder/+merge/222614
Your team Ubuntu Phablet Team is requested to review the proposed merge of lp:~pete-woods/net-cpp/uri-builder into lp:net-cpp.



More information about the Ubuntu-reviews mailing list