Re: cvs commit: apr/include apr_xlate.h

2001-01-28 Thread Ben Laurie
[EMAIL PROTECTED] wrote: fielding01/01/28 03:33:55 Modified:i18n/unix xlate.c include apr_xlate.h Log: Revert last change and solve warning by explicit cast. The need would have been obvious if dependencies were in the Makefile. Revision Changes

Re: cvs commit: apr/time/unix time.c

2001-01-28 Thread William A. Rowe, Jr.
Sorry, here's what changed without win32 cruft tossed in... wrowe 01/01/28 07:30:32 Modified:.apr.dsp apr.mak libapr.dsp libapr.mak dso/os2 dso.c file_io/os2 dir.c filedup.c filestat.c maperrorcode.c readwrite.c

includes

2001-01-28 Thread William A. Rowe, Jr.
The last os2 patch finishes the move of all .h files to arch/ excepting mmap/unix/mm/mm.h, which we all agreed stays put since that is a package private to unix. Brian, please test, beat on me if it breaks anything and I'll revert as you propose. -Before- you beat on me, please look at apr.hw to

Re: cvs commit: apr CHANGES configure.in

2001-01-28 Thread Dale Ghent
On 28 Jan 2001 [EMAIL PROTECTED] wrote: | + *) Replace configure --with-optim option by using the environment | + variable OPTIM instead. This is needed because configure options | + do not support multiple flags separated by spaces. [Roy Fielding] | + Sure it can support

Re: cvs commit: apr/include apr_xlate.h

2001-01-28 Thread rbb
Eh? Why are you casting away the const??? Because, as noted in the earlier commit, the iconv function does not have a const parameter, and the only reason this variable is being used is for passing that parameter. Blame it on whoever created the original iconv prototype for the

Re: cvs commit: apr/include apr_xlate.h

2001-01-28 Thread rbb
size_t iconv(iconv_t cd, const char **inbuf, size_t *inbytesleft, char **outbuf, size_t *outbytesleft); That shows that SUSv2 is a descendant of XPG4 where this specific iconv interface (with const) originated. However, draft 5 of SUSv3 contains this

Re: cvs commit: apr STATUS

2001-01-28 Thread David Reid
Are we not using it as we haven't come across the case where we need it yet? If that's the case then I'll remove my +1 but from the note it looked as if we just weren't ever using it. david @@ -62,7 +62,13 @@ - It ignores the type parameter, so toss it. - The fname param

Re: cvs commit: apr STATUS

2001-01-28 Thread rbb
On Sun, 28 Jan 2001, David Reid wrote: Are we not using it as we haven't come across the case where we need it yet? If that's the case then I'll remove my +1 but from the note it looked as if we just weren't ever using it. We aren't using the type parameter because nobody has had the time to

Re: cvs commit: apr/include apr_xlate.h

2001-01-28 Thread Sascha Schumann
Btw, here is the reasoning for the change: http://www.opengroup.org/sophocles/show_mail.tpl?source=Llistname=austin-group-lid=270 That's a real PITA. How exactly are we supposed to handle platforms this? Linux is getting warnings right now, which I dislike. Why is it a PITA?