Re: compaudit problem

[Alexey Tourbin <at@altlinux.ru>]

[-- Attachment #1: Type: text/plain, Size: 1318 bytes --]

On Tue, Oct 17, 2006 at 08:41:50PM -0700, Bart Schaefer wrote:
> On Oct 17, 11:05pm, Alexey Tourbin wrote: > }
> } On Sat, Aug 19, 2006 at 10:30:33AM -0700, Bart Schaefer wrote:
> } > 
> } >         * 22557: Doc/Zsh/options.yo, Misc/globtests, Src/options.c,
> } >         Src/pattern.c, Test/D02glob.ztst, Test/D07multibyte.ztst:
> } >         Turn on multibyte option by default for MULTIBYTE_SUPPORT and fix
> } >         tests and patterns.
> } 
> } git-bisect says the problem started with change #22557.
> 
> Given that, according to your build script, you compiled both the before-
> and after-22557 sources with --enable-multibyte, the only change in 22557
> that could possibly be related is this one:
> 
> --- Src/pattern.c	25 Jul 2006 09:25:27 -0000	1.37
> +++ Src/pattern.c	25 Jul 2006 18:06:48 -0000
> @@ -343,7 +343,7 @@
>      /* Error.  Treat as single byte. */
>      /* Reset the shift state for next time. */
>      memset(&shiftstate, 0, sizeof(shiftstate));
> -    return (wchar_t) *(*x)++;
> +    return (wchar_t) STOUC(*(*x)++);
>  }
>  
>  #else
> 
> Does backing out that STOUC fix the problem?  What underlying integer
> type is wchar_t on the machine where you're building zsh?

Nope.  But reverting Src/options.c part fixes the problem.
I'm kind of puzzled.

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]