Re: cvs commit: jakarta-tomcat-connectors/ajp/ajplib/test ajp_header.c

2004-07-30 Thread jean-frederic clere
Mladen Turk wrote: [EMAIL PROTECTED] commited: Log: Arrange small things and temporaly modify ajp_read_header(). I really don't understand why is unacceptable to include ajp.h in mod_proxy. For example my compiler is complaining about missing prototypes in proxy_ajp. So either we'll make anot

RE: cvs commit: jakarta-tomcat-connectors/ajp/ajplib/test ajp_header.c

2004-07-30 Thread Mladen Turk
[EMAIL PROTECTED] commited: > Log: > Arrange small things and temporaly modify ajp_read_header(). > I really don't understand why is unacceptable to include ajp.h in mod_proxy. For example my compiler is complaining about missing prototypes in proxy_ajp. So either we'll make another header

RE: cvs commit: jakarta-tomcat-connectors/ajp/ajplib/test ajp_header.c ajp.h

2004-07-30 Thread Mladen Turk
jean-frederic clere wrote: > > > > But the ajp.h is a public API for libajp, so, it should be > included in > > any project using it. > > It has defined > > OK but I want to keep the AJP things as headen as possible in > the proxy_ajp.c so > we need another include. > Sure thing. Only t

Re: cvs commit: jakarta-tomcat-connectors/ajp/ajplib/test ajp_header.c ajp.h

2004-07-30 Thread jean-frederic clere
Mladen Turk wrote: -Original Message- From: jean-frederic clere The ideas was to prevent including of ajp.h in proxy_ajp.c Now ajp_header.c doesn't compile: But the ajp.h is a public API for libajp, so, it should be included in any project using it. It has defined OK but I want to

RE: cvs commit: jakarta-tomcat-connectors/ajp/ajplib/test ajp_header.c ajp.h

2004-07-30 Thread Mladen Turk
> -Original Message- > From: jean-frederic clere > > The ideas was to prevent including of ajp.h in proxy_ajp.c > Now ajp_header.c doesn't compile: But the ajp.h is a public API for libajp, so, it should be included in any project using it. It has defined MT. smime.p7s Descripti

Re: cvs commit: jakarta-tomcat-connectors/ajp/ajplib/test ajp_header.c ajp.h

2004-07-30 Thread jean-frederic clere
[EMAIL PROTECTED] wrote: mturk 2004/07/29 23:32:54 Modified:ajp/ajplib/test ajp_header.c ajp.h Log: Change the casting from void* to msg*. We don't need that, cause the ajp_msg_t is public. The ideas was to prevent including of ajp.h in proxy_ajp.c Now ajp_header.c doesn't compile

Re: cvs commit: jakarta-tomcat-connectors/ajp/ajplib/test ajp_header.c

2004-07-28 Thread jean-frederic clere
jean-frederic clere wrote: [EMAIL PROTECTED] wrote: mturk 2004/07/28 09:34:22 Modified:ajp/ajplib/test ajp_header.c Log: Modify sc_for_req_method. It is exactly the same as Apache's lookup_builtin_method with added SC_M_SEARCH Thanks. Mine was telling: +++ [Wed Jul 28 18:43:56 20

Re: cvs commit: jakarta-tomcat-connectors/ajp/ajplib/test ajp_header.c

2004-07-28 Thread jean-frederic clere
Mladen Turk wrote: [EMAIL PROTECTED] jakarta-tomcat-connectors/ajp/ajplib/test ajp_header.c +rc = ajp_msg_create(r->pool,msg); Can we use Apache coding convention having one space after comma in function params? Oops, I have done lot of those... I will fix them tomorrow. It won't IMO have

Re: cvs commit: jakarta-tomcat-connectors/ajp/ajplib/test ajp_header.c

2004-07-28 Thread jean-frederic clere
[EMAIL PROTECTED] wrote: mturk 2004/07/28 09:34:22 Modified:ajp/ajplib/test ajp_header.c Log: Modify sc_for_req_method. It is exactly the same as Apache's lookup_builtin_method with added SC_M_SEARCH Thanks. Mine was telling: +++ [Wed Jul 28 18:43:56 2004] [debug] ajp_header.c(239

RE: cvs commit: jakarta-tomcat-connectors/ajp/ajplib/test ajp_header.c

2004-07-28 Thread Mladen Turk
[EMAIL PROTECTED] > jakarta-tomcat-connectors/ajp/ajplib/test ajp_header.c > > +rc = ajp_msg_create(r->pool,msg); Can we use Apache coding convention having one space after comma in function params? It won't IMO have a chance to get into httpd tree otherwise. I wanted to do that for JK2,