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

RE: [Patch] RE: Windows drive letter check fails on lower case current working drive

From: Bert Huijben <bert_at_qqmail.nl>
Date: Wed, 19 May 2010 21:48:24 +0200

> -----Original Message-----
> From: Bert Huijben [mailto:bert_at_qqmail.nl]
> Sent: woensdag 20 januari 2010 11:31
> To: 'Branko ─îibej'; 'William A. Rowe Jr.'
> Cc: dev_at_apr.apache.org; 'Philip Martin'; 'Bert Huijben';
> dev_at_subversion.apache.org
> Subject: RE: [Patch] RE: Windows drive letter check fails on lower case
> current working drive
>
> > -----Original Message-----
> > From: Branko ─îibej [mailto:brane_at_xbc.nu]
> > Sent: woensdag 20 januari 2010 10:17
> > To: William A. Rowe Jr.
> > Cc: Bert Huijben; dev_at_apr.apache.org; 'Philip Martin'; 'Bert
> Huijben';
> > dev_at_subversion.apache.org
> > Subject: Re: [Patch] RE: Windows drive letter check fails on lower
> case
> > current working drive
> >
> > William A. Rowe Jr. wrote:
> > > On 1/19/2010 5:19 AM, Bert Huijben wrote:
> > >
> > >> The patch was written on the 1.4.x branch but I svn switch'ed it
> to trunk
> > for easy application.
> > >>
> > >
> > > I would suggest one bit of alternate code that is a bit more
> condensed, any
> > > concern?;
> > >
> > > +static int same_drive(const char *path1, const char *path2)
> > > +{
> > > + if ((path1[0] < 'A' || (path1[0] > 'Z' && path1[0] < 'a') ||
> path1[0] > 'Z')
> > > + || (path2[0] < 'A' || (path2[0] > 'Z' && path2[0] < 'a') ||
> path2[0] > 'Z')
> > > + || path1[1] != ':' || path2[1] != ':')
> > > + return 0;
> > >
> > > + /* Once in the domain of ASCII alpha, compare these case
> insensitive */
> > > + return ((path1[0] & 0x1f) == (path2[0] & 0x1f));
> > > +}
> > >
> > Whatever you do, those last Z's should be z's or it all breaks.
>
> No, problem... I initially wrote a more condensed form myself and just
> made it a bit more readable later.
>
> But please fix that 'Z' ;)

        Ping,

Subversion is trying to get Subversion 1.7 test releases out by this summer, and we really need this fixed (or a workaround) before this release.

        Bert
Received on 2010-05-19 21:49:17 CEST

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