Zsh Mailing List Archive Messages sorted by: Reverse Date, Date, Thread, Author

Re: New D01 test failure

X-seq: zsh-workers 26665
From: Bart Schaefer <schaefer@xxxxxxxxxxxxxxxx>
To: Zsh hackers list <zsh-workers@xxxxxxxxxx>
Subject: Re: New D01 test failure
Date: Mon, 02 Mar 2009 21:18:35 -0800
In-reply-to: <20a807210903021619h2b045423s641d4c69aa75cbfd@xxxxxxxxxxxxxx>
Mailing-list: contact zsh-workers-help@xxxxxxxxxx; run by ezmlm
References: <20a807210903021619h2b045423s641d4c69aa75cbfd@xxxxxxxxxxxxxx>

On Mar 2,  7:19pm, Vin Shelton wrote:
} Subject: New D01 test failure
}
} I'm seeing a new failure in the prompt expansion test.

This seems to do it.  The only bug was that (strip) should have
been (!strip), but I reversed the order of the || contitions to
match all the other uses of (!strip) in the same case statement.

Index: Src/utils.c
--- ../current/Src/utils.c    2009-03-02 17:57:29.000000000 -0800
+++ Src/utils.c 2009-03-02 21:15:36.000000000 -0800
@@ -2460,7 +2460,7 @@
                return -1;
            switch (*fmt++) {
            case 'd':
-               if (strip || tm->tm_mday > 9)
+               if (tm->tm_mday > 9 || !strip)
                    *buf++ = '0' + tm->tm_mday / 10;
                *buf++ = '0' + tm->tm_mday % 10;
                break;
@@ -3090,6 +3090,7 @@
     for (t0 = 0240; t0 != 0400; t0++)
        typtab[t0] = IALPHA | IALNUM | IIDENT | IUSER | IWORD;
 #endif

References:
- New D01 test failure
  - From: Vin Shelton

Messages sorted by: Reverse Date, Date, Thread, Author