[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] Re: apr_strnatcasecmp vs. strcasecmp

From: Paul Burba <paulb_at_softlanding.com>
Date: 2006-02-21 18:44:34 CET

Mark Phippard <markp@softlanding.com> wrote on 02/21/2006 12:21:38 PM:

> Paul Burba/SoftLanding Systems wrote on 02/21/2006 12:14:21 PM:
>
> > Thanks for the info. Mark and I talked about this offline and agree
> that
> > your approach is probably the right way to go. So here is a patch to
> all
> > files that call strcasecmp() and/or strncasecmp() and don't already
have
>
> > #define APR_WANT_STRFUNC and #include <apr_want.h>.
> >
> > You are contractually obligated to approve it now since it was your
> idea,
> > right? ;-)
> >
> > Thanks,
> >
> > Paul B.
> >
> > [[[
> > Support platforms with old BSD-style string.h and strings.h headers.
> >
> > OS400 is the only platform this is currently known to affect.
> >
> > * subversion/libsvn_ra/ra_loader.c
> > * subversion/libsvn_ra_serf/serf.c
> > * subversion/libsvn_ra_serf/update.c
> > * subversion/libsvn_ra_serf/util.c
> > * subversion/libsvn_subr/opt.c
> > * subversion/libsvn_subr/subst.c
> > Define APR_WANT_STRFUNC and include apr_want.h.
> > ]]]
>
> I noticed you did not delete the #include for string.h as part of this
> patch. Wouldn't that be the correct way to do it?

Mark - Indeed it is, corrected below.
 
> I am also adding Justin to the cc: list since this included changes to
> ra_serf.
>
> Mark

[[[
Support platforms with old BSD-style string.h and strings.h headers.

OS400 is the only platform this is currently known to affect.

* subversion/libsvn_ra/ra_loader.c
* subversion/libsvn_ra_serf/serf.c
* subversion/libsvn_ra_serf/update.c
* subversion/libsvn_ra_serf/util.c
* subversion/libsvn_subr/opt.c
* subversion/libsvn_subr/subst.c
   Define APR_WANT_STRFUNC and include apr_want.h. Remove
   #include <string.h> if present.
]]]

_____________________________________________________________________________
Scanned for SoftLanding Systems, Inc. and SoftLanding Europe Plc by IBM Email Security Management Services powered by MessageLabs.
_____________________________________________________________________________

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Received on Tue Feb 21 18:45:08 2006

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.