DomainKey-Signature: a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:from:to:subject:date:message-id:references
	:in-reply-to:content-type:content-id:content-transfer-encoding
	:mime-version; q=dns; s=default; b=irXTyOtYc03tZy8RVvHiWrG/HWnBv
	J0m6DCPABDbqrMNqG4Lh9WRBMfsMuZQ+aNSQr/L6g0Xy4lnhTlRuVVGRmu+jqmtM
	h7abXgZQgqQ4LGc9RKTdNklyjgyoHlapBrKpjcde/V5NWceOD11P5NKyGEKmgLEd
	C5WZR8q2Pma/44=
Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm
Sender: cygwin-owner AT cygwin DOT com
Mail-Followup-To: cygwin AT cygwin DOT com
From: Ken Brown <kbrown AT cornell DOT edu>
To: "cygwin AT cygwin DOT com" <cygwin AT cygwin DOT com>
Subject: Re: Solved. Odd, is it not? mkdir 'e:\' cannot be undone by rmdir 'e:\' ...
Date: Sat, 21 Sep 2019 21:02:37 +0000
Message-ID: <18fad38d-8b50-959c-1b4c-4b3080bd8c62@cornell.edu>
References: <8e310c59702b57501ca2fd80edf19d2e AT smtp-cloud9 DOT xs4all DOT net>
In-Reply-To: <8e310c59702b57501ca2fd80edf19d2e@smtp-cloud9.xs4all.net>
user-agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0
received-spf: None (protection.outlook.com: cornell.edu does not designate permitted sender hosts)
Content-Type: text/plain; charset="Windows-1252"
Content-ID: <D3FC07BB58658349BB70C9B4BEB7044A AT namprd04 DOT prod DOT outlook DOT com>
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-userprincipalname: XE4HMJCmfhMfovTybNn0pDfsan+N/2boAyRtotZnEAEw0iEVN3R2rnAnwUSF1wEnkifZKxrCeOxSOAiLR37/Hw==
X-IsSubscribed: yes
Content-Transfer-Encoding: 8bit
X-MIME-Autoconverted: from quoted-printable to 8bit by delorie.com id x8LL3EEP016832

On 9/6/2019 5:53 PM, Houder wrote:
> However an exception can be made for e:/ (or e:\), as follows:
> 
> --
>        char flag = '\0';
>        // strip trailing dirsep's, while remembering the last one
>        if (isdirsep (dir[strlen (dir) - 1]))
>          {
>            flag = dir[strlen (dir) - 1];
>            /* This converts // to /, but since both give EEXIST, we're okay.  */
>            char *buf;
>            char *p = stpcpy (buf = tp.c_get (), dir) - 1;
>            dir = buf;
>            while (p > dir && isdirsep (*p))
>             {
>              flag = *p;
>              *p-- = '\0';
>             }
>          }
> 
>        // reattach dirsep in case of x: and flag != '\0'
>        if ( (strlen (dir) == 2)
>             && (dir[1] == ':')
>             && isalpha (dir[0]) && flag != '\0' )
>          {
>            char *buf = tp.c_get ();
>            buf[0] = dir[0];
>            buf[1] = ':';
>            buf[2] = flag;
>            buf[3] = '\0';
>            dir = buf;
>          }

I think you can simplify this by eliminating the second part and changing the 
first part to the following:

         char sep = dir[strlen (dir) - 1];
         if (isdirsep (sep)
           {
             /* This converts // to /, but since both give EEXIST, we're okay. */
             char *buf;
             char *p = stpcpy (buf = tp.c_get (), dir) - 1;
             dir = buf;
             while (p > dir && isdirsep (*p))
               *p-- = '\0';
             /* Reattach dirsep in case of "x:". */
             if (p == dir + 1 && *p == ':' && isalpha (dir[0]))
               p[1] = sep;
           }

Ken

--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple