Tommi Mäkitalo <tommi@...> writes: > > Hi, > > fine, that you found the regex code already. I haven't come to it to answer > your previous patch. If you look at it, it is quite different from your > suggested patch. The reason is, that I try to keep cxxtools compatible to > previous versions and since the url was not a regex it should not be now. > Another reason is just performance. Your patch compiled the regex on every > request. This takes time. It is better to store the regex class somewhere. > And > another hint: you don't need to free the regex explicitely. It is done by the > destructor automatically. > > And now to your new question. The problem is, that the body in the request is > not meant for deliver the body to the responder. That sounds strange, I know. > And therefore I changed it . > > The Request-class is used in http server as well as in client. The body is > used, when I use the Request-class as a client. But indeed it does not hurt > to > use it in server code as well. Previously a responder was asked to read the > body from the stream by overriding the method "std::size_t > Responder::readBody(std::istream&)". The default implementation just > discarded > the body. > > Tommi
Thank you very much! But I can't compile rev. 1230: serverimpl.cpp: In member function ‘void cxxtools::http::ServerImpl::start()’: serverimpl.cpp:104:5: error: ‘::signal’ has not been declared serverimpl.cpp:104:14: error: ‘SIGPIPE’ was not declared in this scope serverimpl.cpp:104:23: error: ‘SIG_IGN’ was not declared in this scope Is this line really required by cxxtools? nice regards Michael ------------------------------------------------------------------------------ Simplify data backup and recovery for your virtual environment with vRanger. Installation's a snap, and flexible recovery options mean your data is safe, secure and there when you need it. Discover what all the cheering's about. Get your free trial download today. http://p.sf.net/sfu/quest-dev2dev2 _______________________________________________ Tntnet-general mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/tntnet-general
