From: Dean M. B. <mik...@gm...> - 2008-10-21 10:27:52
|
Hi Kim, On Tue, Oct 21, 2008 at 6:22 PM, Kim Gräsman <kim...@gm...> wrote: > Hi guys, > > Rev. 114 in the uri branch contains enough directive support to pass > the test Glyn added (i.e. directives path and query_param). > Nice, I'll try and take a look as soon as I get time to do that. > I've gone back and forth between fusion::map and regular members, but > after Dean's suggestions that there are indeed benefits in the fusion > map, I've gone back to the original layout. > > There were also some implicit assumptions that the URI was for HTTP, > with path normalization, etc. I've removed that, so that clients will > need to translate empty path to "/" if necessary, and the same with > the port number. > Okay. Can you try integrating (i.e. merging your changes) with http_integration to make the request object use the refactored URI implementation? > Hope that helps, Sure does, thanks Kim! -- Dean Michael C. Berris Software Engineer, Friendster, Inc. |