From: Stephen D. <sd...@gm...> - 2005-03-08 05:21:45
|
It looks like the change was added to cvs-HEAD ~6 weeks ago, so this probably wont find it's way to 4.0.10 at all... I was just thinking we should keep parity with AOLserver here. The new driver code promised for AOLserver might depend upon it and it might start to get messy when we merge with 4.1 if we have to rename everything. On Tue, 08 Mar 2005 00:06:19 -0500, Vlad Seryakov <vl...@cr...> wrote: > Done except naming, it is different in AS 4.1 CVS and 4.0.x, does it > really matter much? But i am not insisting. > > Stephen Deasey wrote: > > How about removing the comment, marking the function void, and > > changing the naming to match the addtions to AOLserver? > > > > > > On Mon, 07 Mar 2005 23:53:35 -0500, Vlad Seryakov > > <vl...@cr...> wrote: > > > >>Oops, i was thinking to retern previous status but then change my mind, > >>comment left, i will correct it. > >> > >>Stephen Deasey wrote: > >> > >>>The comment for Ns_ConnSetResponseStatus() says that it reurns the > >>>previous status. It actually returns the current, just-set status. > >>>So does ns_conn status. > >>> > >>>I don't think it's neccessary. The other simillar functions in that > >>>file have no return value. (Ns_ConnSetUrlEncoding(), > >>>Ns_ConnSetWriteEncodedFlag()...) > >>> > >>> > >>>Added six weeks ago to AOLserver cvs: Ns_ConnSetStatus(), Ns_ConnGetStatus() > >>> > >>>http://cvs.sourceforge.net/viewcvs.py/naviserver/naviserver/nsd/conn.c?r1=1.1&r2=1.2 > >>> > >>> > >>> > >>> > >>>On Sat, 05 Mar 2005 13:22:31 -0500, Vlad Seryakov > >>><vl...@cr...> wrote: > >>> > >>> > >>>>I know, it is just the core of the update to be approved > >>>> > >>>>Zoran Vasiljevic wrote: > >>>> > >>>> > >>>>>On Saturday 05 March 2005 18:36, Vlad Seryakov wrote: > >>>>> > >>>>> > >>>>> > >>>>>>Patch is attached, if everybody is Okay with it i will commit it > >>>>> > >>>>> > >>>>>Do not forget to update include/ns.h > >>>>>and put the prototype there! > >>> > >>> > >>> > >>>------------------------------------------------------- > >>>SF email is sponsored by - The IT Product Guide > >>>Read honest & candid reviews on hundreds of IT Products from real users. > >>>Discover which products truly live up to the hype. Start reading now. > >>>http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click > >>>_______________________________________________ > >>>naviserver-devel mailing list > >>>nav...@li... > >>>https://lists.sourceforge.net/lists/listinfo/naviserver-devel > >> > >>-- > >>Vlad Seryakov > >>571 262-8608 office > >>vl...@cr... > >>http://www.crystalballinc.com/vlad/ > >> > >>------------------------------------------------------- > >>SF email is sponsored by - The IT Product Guide > >>Read honest & candid reviews on hundreds of IT Products from real users. > >>Discover which products truly live up to the hype. Start reading now. > >>http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click > >>_______________________________________________ > >>naviserver-devel mailing list > >>nav...@li... > >>https://lists.sourceforge.net/lists/listinfo/naviserver-devel > >> > > > > > > > > ------------------------------------------------------- > > SF email is sponsored by - The IT Product Guide > > Read honest & candid reviews on hundreds of IT Products from real users. > > Discover which products truly live up to the hype. Start reading now. > > http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click > > _______________________________________________ > > naviserver-devel mailing list > > nav...@li... > > https://lists.sourceforge.net/lists/listinfo/naviserver-devel > > -- > Vlad Seryakov > 571 262-8608 office > vl...@cr... > http://www.crystalballinc.com/vlad/ > > ------------------------------------------------------- > SF email is sponsored by - The IT Product Guide > Read honest & candid reviews on hundreds of IT Products from real users. > Discover which products truly live up to the hype. Start reading now. > http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click > _______________________________________________ > naviserver-devel mailing list > nav...@li... > https://lists.sourceforge.net/lists/listinfo/naviserver-devel > |