From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <zsh-workers-return-36421-mason-zsh=primenet.com.au@zsh.org>
Received: (qmail 13878 invoked by alias); 4 Sep 2015 12:36:08 -0000
Mailing-List: contact zsh-workers-help@zsh.org; run by ezmlm
Precedence: bulk
X-No-Archive: yes
List-Id: Zsh Workers List <zsh-workers.zsh.org>
List-Post: <mailto:zsh-workers@zsh.org>
List-Help: <mailto:zsh-workers-help@zsh.org>
X-Seq: 36421
Received: (qmail 3725 invoked from network); 4 Sep 2015 12:36:06 -0000
X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on f.primenet.com.au
X-Spam-Level: 
X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00 autolearn=ham
	autolearn_force=no version=3.4.0
X-AuditID: cbfec7f5-f794b6d000001495-7d-55e990332a7e
Date: Fri, 04 Sep 2015 13:35:42 +0100
From: Peter Stephenson <p.stephenson@samsung.com>
To: zsh-workers@zsh.org
Subject: Re: invalid characters and multi-byte [x-y] ranges
Message-id: <20150904133542.4673f880@pwslap01u.europe.root.pri>
In-reply-to: <20150904124719.36ad9457@pwslap01u.europe.root.pri>
References: <20150902230711.GA4967@chaz.gmail.com>
 <20150903100037.6e6ac852@pwslap01u.europe.root.pri>
 <20150903100943.GB7821@chaz.gmail.com>
 <20150903151811.557a40ec@pwslap01u.europe.root.pri>
 <loom.20150904T125242-645@post.gmane.org>
 <20150904124719.36ad9457@pwslap01u.europe.root.pri>
Organization: Samsung Cambridge Solution Centre
X-Mailer: Claws Mail 3.7.9 (GTK+ 2.22.0; i386-redhat-linux-gnu)
MIME-version: 1.0
Content-type: text/plain; charset=US-ASCII
Content-transfer-encoding: 7bit
X-Brightmail-Tracker:
 H4sIAAAAAAAAA+NgFjrELMWRmVeSWpSXmKPExsVy+t/xy7rGE16GGjybzGdxsPkhkwOjx6qD
	H5gCGKO4bFJSczLLUov07RK4Mnp3HWApmMtZceQWSwPjevYuRk4OCQETiaWLzzBC2GISF+6t
	Z+ti5OIQEljKKLG67ygzhDODSWLKpCksEM5WRon/Xc+B2jk4WARUJaas4wfpZhMwlJi6aTbY
	JBEBcYmza8+zgNjCAtYSP2+sBLN5Bewlzu+4ygTSyingIPGzTw1i5Eomib1TZrKC1PAL6Etc
	/fuJCeIie4mZVyCu4xUQlPgx+R7YHGYBLYnN25pYIWx5ic1r3jKD2EIC6hI37u5mn8AoNAtJ
	yywkLbOQtCxgZF7FKJpamlxQnJSea6RXnJhbXJqXrpecn7uJERKyX3cwLj1mdYhRgINRiYf3
	5I8XoUKsiWXFlbmHGCU4mJVEeKcGvgwV4k1JrKxKLcqPLyrNSS0+xCjNwaIkzjtz1/sQIYH0
	xJLU7NTUgtQimCwTB6dUA+OKeftnGZ+tXC/8LePC8r4qvQ+lK2yeq4ha1fQHy0ofmlW74TRP
	mpSPyspzPA7/N+k/9u5ifr/N6YpYbYHg3Pe2lpvnfkiW92Oq1LvCEnb3kLhvY0duubXm4kN5
	mf9vxO9k3hL5a5vkrNPbohekPj8bEGVzTuX87+mrV+eLmNdl95+b87Cv+ZISS3FGoqEWc1Fx
	IgCV0E3iVQIAAA==

On Fri, 4 Sep 2015 12:47:19 +0100
Peter Stephenson <p.stephenson@samsung.com> wrote:

> On Fri, 4 Sep 2015 10:53:14 +0000
> Ismail Donmez <ismail@i10z.com> wrote:
> on <p.stephenson <at> samsung.com> writes:
> > This seems to break glob tests:
> 
> I missed that --- it looks like there's something funny with the (#i)
> flag.  That may mean there's a pre-existing funny since it shouldn't be
> encountering anything looking like invalid characters.

It was just stupidity.  I changed a line that referred to an ASCII
character rather than an invalid multibyte character.

> Looks like the alias test needs updating for the new report format
> (presumably trivial).

It waqs just stupidity.  I hadn't recompiled after pulling that change.

Can't get the staff.
pws

diff --git a/Src/pattern.c b/Src/pattern.c
index 7457cbd..b4ba33e 100644
--- a/Src/pattern.c
+++ b/Src/pattern.c
@@ -1929,7 +1929,7 @@ charrefinc(char **x, char *y, int *z)
     size_t ret;
 
     if (!(patglobflags & GF_MULTIBYTE) || !(STOUC(**x) & 0x80))
-	return WCHAR_INVALID(*(*x)++);
+	return (wchar_t) STOUC(*(*x)++);
 
     ret = mbrtowc(&wc, *x, y-*x, &shiftstate);