ZLE 8-bit patch

ZLE 8-bit patch

[Zefram]

-----BEGIN PGP SIGNED MESSAGE-----

I've done the work to make the main part of ZLE 8-bit clean.
The patch is included below, compressed because it's 62kB.  I made
it after applying the recent i-search patch, among others.

The patch also makes a couple of other minor bits of zsh 8-bit
clean.  Things like the history builtin, which I did because I had
made the history mechanism 8-bit clean.  It also fixes a couple
of minor ZLE bugs I discovered in the course of making the patch.
The patch doesn't touch the prompt code, or anything in zle_tricky.c.
I intend to do those soon, but what I have here is a useful change
even without them.

The i-search code is now a real mess.  I just changed it the
minimum necessary to make it work as well on binary data as it
did on printable characters.  There's also the problem that it
doesn't indicate failing properly, which it really should do in the
minibuffer prompt.  Someone should review/rewrite the code properly.

 -zefram

begin 644 zle_8bit_patch.gz
M'XL("+7\DC$``VYE=P#L77EWVT:2_YO^%"WMB\0#E`%>XA%EGN.5Q]JQ9:\M
MSQZ.1P\"FR+&),`0H&0IX^^^5=4G0)"Z*"4[;Y1$(M'=U56_.KKZ0HZB(?_6
M9V>+<)*&T5[PK%JMFF\EK]?K/'?;S]T>:WC]9K??[)2\O>ZS>KU>4,MSF=ON
MNRVHA73L'Z+;\'H=I]%P&XP>,59B\!..6#D(V,$!VPF"TU$X3](*EI5F\S!*
M1^5M5C_9K@SPT7?&)PE_MB4:4OFOBSCEPW)"%?#G._R5'Y]7V22.9RP=S^/%
M^9B-)OYYPBZ?Q\R?PX=T[*?PB;.$IXZ@%D;G\)A/D:DD9M7G)*GA&[[5'\RW
M8/EZMDB3<N*P)!W&B_2Q^"_40Z/1!GF:GM*#1C,<@4CUGY+%63`=.FQWLJOY
M$N+6?QK/_+3"?D.QC*"OV0]#MNTPJA`MIM!J2Y9+!>FFQ7)&\24RSNK?V-[>
M'N#,^+>4@W$.61!/9Q.>AG'$9OX\U4K10DBE/)(0MK94TQN4=C=A"C74=%T'
M?G6TIR#GT/\D3%*VPUZ^>_O^],W1QY,*V]DA,]P29HCTXZ2B;#1;..0C?S%)
M\VV$Z4HX",5%&HS]>7F7[0H,BKVM])WP4Y5_B78M/$A%6@KE-[^[%*M];ZTT
MQ3IJ[H-TK:;M1:#]HQ$Y*@O8V$^8'Z&S+J8\4CXJ_!84;TPR2>=+UOA#0+:X
M[$E8V;8[LCT!MV+(\HA'8BB#8SHO=`ABK!"Y5J/AP*]>9AR8\W0QCYA'R@`6
M+VHU0VU+"7,RYA#Q0I"#3?UOX70Q9?&(I9<Q&X-1Q?,KELQX$(Y"/D_V&'N1
ML@GW$Q`UI8:7X63"SKBD!W!,8@B9?D+^JBB0>4(7<<3KPQ"02L!?_<E>%J,J
M\BA1NK[TYU'D3\O;HP!`VD[C&-B+KC34"3P]_O3FC<-<86U&6E*=1L0>89X$
MD:JDQ*H*G#O#(G\>!9UB^^GV`*V>CHZ_J8[!F"'F(PF#SRL,#"P%`5+L%OP;
MD%(R09@_XW,0X9B?^VEXP>63!&@-P\!/.6CA`K[SH:0HRF&@S9E#&3IF!\Q/
MXQ#BHPI$-.!`P8\@F5*KJ!@LYL@$JPF.A14HN:05;%(N2\\K1;2%,S^;%+-0
MG9V&Y\"O?:5.10&$A)@",8;,$`8%,)JA8$=G8"-,P&3P`5X@-)6B,.`C%>%'
MPJ`HL%-4UX_(C41BAA5&,M*-P$!_2'Z)P"1QF".]:`:E7C;`X/6F&"Q"M-WJ
M-9UVNVTC*OF-I*U@IIB$Y^"_1K_2Z^H8E'`(AV$%F[RG0<('(=`R'#F&HG`A
MF&4<I7X8)2+(`#4</OT@I6"#J9H@0I\NXG`(?S)I(=1F51QNH.`W^(\LAQ[.
MTOE`/L#^PXB:@$FY`T%52A2,>?"5C>(YX]-9>B79%'T+)92WJG)`JYL!;7<7
MD_.ZBC6JJ^\K:"^+!QR1VO/]8>TR,,]0[?.!$(3!KUH-A21?"26Y,A96Z#$V
M/8_!E6&0EJ59.4%S!4S@%&`RL7HO&)+KEG8MFJ:COH8*68/I1/G#R__\].[D
M\&-%H@;]+Q((%$$]2:\F7-A`4K\,4["DNOA&DB,BE^-XDL7%RJMV=P7LZV`2
M?>JAA&H=,&J+!074L"YZ"FK;U(]V,4D4$KW\^/H_/AW_Y>A0"9:C](M%23TE
M[=#3[X5"&`_-H/1S#$!S"RER&Q_-'Q$19BJ]2*"42)CN"8HH`LLJ2R]13VQA
M#%B"B*AYX(I'WU?`L0)K6VJRZ*V[]^Q9/:NQYH[JR_:RQ+/1GW)MT1O9N>0W
MK_.L<I^IN8P(J7IL/A+K)N>3^$PNFHB/!2LF^R5OKT$T<E7$<HFWWV_T"J-X
MQ^LYG6;7GE[("!U,9P.=^6-L^"">H_/%\R&?JPQ06I>#P6.TB"#E.ELD8<23
MY>`,41%^@P%"&Z`O(W/5=V0XKIXMA^C`8=6A%5$"L-OJF0(/A_^JK[Z1:8-%
MTXI)=8B:K08#%M1J#AN">6?#%"0:P%"EO(APD,*),W18@69U5E@05/281<'2
M'PXI'\:L"&(J`D/I*XUU5$4N(Q#(C59&JQ`8QZE_-N%2M=9WH]]]UFCT&ZU^
MPP7]XC\MHE=45RO:6[$NUL1ILLYF41,0G2%A1CV`.A9!2G2C>,B%2I8?GE4H
M7]RR,33*W"E#0EZI_P0T*P[#;V?R6T4;$N'V,?"%&2%9&N)\ZH613`S3&_Z-
M!PL8B!.H"C85P'Q`UC[S,8>,!0&Q6%55"P$DH$R?-B;@M9!.R^8P(]ACRU6D
MR%[#=7KMENVQ+R8A3KK!%<KTL0)R#TQ&AI%(9"1$=H>]_W!T?')Z_.+MX;OC
M-_\C,T8[8_*->,NK%3J983I\K>_HOUX?'K\\//UX]/;]F\/BWE+^+=UP=Q#,
M;Y`,V\-WU?K%FZ,7'T___.;=SR_>Z`F&RN1$&(8L"":4"#'H"EQ>+KVJ]#G;
M(%?O283^Z^&'G]]]7`'RO007DWXAO!")HNP:R:E%E*O[,.G1OY7=6^M.&[%[
M>Y7)EQYN+]D]COWG>D4P'JE;XP<W2?KD_O!T(&3]8I-`;,X_-H!&X7C1:3F]
M_>QVF&(%W&H[N[\@AKX5@8.*%!,'8L6[H+KV[$Q]Q;1>2E:,V:N1-S%V@^96
M,7@3M*L9+4+4\_9[CN?U&O88?.Q/^7`8SEF$*6E9?7WP4!P-[SH6+^O$IG$;
M'6)]Y/UF'1HHK+"\22ALS4DQ[A$BEJV@B-9MK$="<TOKD;G^(@TGB<SSY>=,
MWN[V/?BW5U+Y?5&==K\)=9K%N7T7]XY[;JM@&C>*,DDJS4TP-]["Q';"([4Z
M)C)?J9HH99.!TK&DWMRW=;Q,O88I\*=HRE-_=$7)+NZPX=9Q9I*(JSDL,['2
M"WE`@E79MO^+N\V2>)Y"F!Q!$3S9MI>*U_R`=#7!"7U".6LJ@<_TB4N*.%-+
M/(<5%S0JT/0W9$D"$D#5H#$0'>#/Y3B<\#*0H&DF_,$Y)[2C5J7$PXT4_-`0
M'X0QZM:X8E@./)R^)EX%.RO1-UP8%3744E-`E'.3T;<`LVE4K=6@_[^Q9F.0
M(=]@@B51LU%,OK&6?$.2;VCRI@?!64!0J>T4MX#]'W-ULDQ8!?3\N])ASIIH
MF90FV3PZ!R.*1WJ]>$]J?HV%*[T6VWFA5_5P\[_7;!9XU9"O]"JQB2^<RF&O
MCMX<TMHSA_B;ZSHP+M9K=Z&K3K?`Q:RNED&1.P,WX6!X4C#<AK-B4)H]X+0E
M0XU@?K_E--VN/B112L0>(N51(]IS"&BVC3UEMF^DB(?O7MD>8I6X:PPB)[O#
MAF$RF_A7M.NPO(@-O0]A'GZU5QPES'[)6IQR,2$?#AC$`QD`:`$U00?1SH35
M`NE/VIU*R@=T:2+BAHP?X$)BO1TYI#$FJ.30K#T&FG?:@5G&,W.DY7:(VM5F
M`PMF:UO&R6"^=F?&!$+T>A-D!)JX-Z-A7$7R-ALRHIE85,<E=5Q0'[`9;L34
MEC9B*EI92]LP-@<W;<-DM*JW(9>%L7=?%!R%NR^U!^Z^&#L-:''[-@9:4@.H
M9$"QEW.:TK+'V"-'R7(9?*[<1E$+[`V,FMC`6,=G$:/D2F9`N\7&P7)?O]RE
M+R/`BOB)C7,-O]]+"99HMA7<<W?I]U2JM3>DH+U)U7D(\WM6-=HYLA0)80,4
MN7UGJU%,9K>P-L*FEV/S#M:9Z[O00I?ZSN!1;)CKK/J[#HH9'&X#PY*],SG&
M9=(E5V1+<N9U/>%[8TI<Z)/96>DRM]EOP;]M/>O*UL`Y5Z??:/2;K<)$J--R
MS*+*<W7>!9/2">Y]J62$RZ,:__OF\+]/#C\<TW@63:(@E0=MH.DTAA%/M,(M
M-0BO9XO1"(A=\MT+B++C^#+"X!]'`1?S)SH)LIC/.2:PN.6$=3#_`X*@?A>=
M)%5'MF`PP3HPA3K`_!_+?!;Q2]$2C^O]?0$?SCB/<!JK]B"28`Y/),5Z`YO-
M^6C.DW&Y`LWY,,&5/&RPF%D$]PKDE=QA\4"ES'*/`P:H&,8GC",DV<1/4ON0
M"VV?`=3ZO.'#H'[[<*CIC%-5(\[N#;BBM`)X=GO<%279ZT-^-JJ\XL6SEN.9
M.170$ONFD59%EH7L[+R*%<6T234-``G3LFY:RNWC10JE`WW0:,MHRVZJ$UQ(
M@/FO"X[ZUP:6I4NS1GAN41;'Z/DH9;-Y#&,C9=3S\'RL'V2%DK+,H,1AU3G^
M-39*>/C23E6^J<[<E6U[C".(GT64Y8$X#9:FFZ1^ND@$Y:*&HERW8P(Q5YBP
M/%?.<>.;X]9W`MB%W]!?D#;'6815#\^?+YL3M14-!W()L>UXYE#JO0WB=]2!
MPV@B%2Y%H-MKPR%VPS11JQLW::>6\U)1-'D2M17Z=1/4V&I;?DV![Q(Z^`93
MD%1/:+[R*[6PIWK$8\!X5<$'&.%Q!'\+6!"UB9B&$9>(;6V88\21V'"'@@3F
M++,X"4DR-8%)XQE50@(R[DIK`R2#K_`8&-AB2\$0"[$31WP,$AMP$5E`9%SU
ME$8S#8?#"1<(3WFTR,%,'M#T("1V+0]X"NCP3/?]<5-#3@%V=QB!"D:;30-<
M:*MM`+QCVZHB+<YG"%23,8?9-I[2(%H!S+W!?4802S!5+.`]C.`YUC<84PRY
M"-%_H_,5`0V;7H3!^!Q*DVM'?Y[A^4KZ@J/M8#D<J(H&H5F,Q"XI$8%>83SC
M\Q3RRR$D,WC@'KD!=4-Q4:YT$4*#,WX>1D:`,+K$BQN)#Y9'G_@W/T@G5T8)
M8I.DC*'KPI\LN#R]K_*KH^-3#'8T0SSCD_BR4I0ABFZ,(,=Q.K;6\?:TJ[0@
M>VC;KO*DFG/(#\S3.9^!H:T(F?^/=6J[J/#S)]!OX7*SAY<B.X79(@==R%$:
MT&!E5,TB&F)N/(>XQ9-\;I3/''QK3";:*HK)JRPRJ]W^5H@K:A8JRF0T4XA<
M!!#!L`<,8'56I.AD'HSQ`.XR:2KAH&BS0>#VG(97G"1M%(?:.DG$WXE8K'P8
M8@_`H]!.6DVGT=:S"KP*.8^H-9_Z07(61L//7QSI#?I+,!V*+TM]^I,42U+_
M['.CW?DB6?ZW(1\AY'_Y<'3\YY<GK`N^D1/C*ZZ%?985ENE2,<Q?!\LM"9_/
M3>PL7W81HGYT,?$"(]U4[]E<G_I!P&<IC-SC>#)TY(%(\)U]O2OS*)C0/L++
M1?K\*UWK$M:07LTX)`$G)B^`X,$G(XP<8),<(@T0\B'=&/JI[ZB-WRA.\8)6
M'=B<0C',*H$(9,/BA(U,BX-X$>$<5TQ`%V#=Y`DB4A&9EY].?O[TZM7A!XA+
MM)G@1Q"ZA.:&(D2MS5#4;H8\'2IF>2A$9INB2C%:/$C":WZ*-]ZB@5V%6*0=
M%[DBC[@`'\N4JR_51UE3&9J1Y<W1\2&DA6+R36-0?VGJ*A;E]>H&@FOMS>`5
MFRF"GITPUS(^O\2:FN0J(H=H.G)=V-:YV0BXR4^6NKB#RXCQZZ\AY;E#2Q*\
M=.C6&VT:O^@(+E60A6S;IU-GUV*6)8>T1J?>;)D&M*BDVXCLDVU[U+!79#%B
M@K!.,NG2[2^&=ZBL5TC$L6"Z("<6&6[AV&9=\Q2S9'FHQ'S-W0UHM?MMK^3M
M==0*9[X:+G-V^VZOW_`*`VNK[;0S5_C5<BNNX:I9+9T"$)-<=1LS&<TY+ULS
M7W'&S7K`#G!_?KB8E3^=O/OX\GV9:F4.?>&/KEX_8-/%)!U8_6^5$Z#RZSE,
M?2X@!=+\B"N#*+%@W]I4WR3[8O<TPSZ,D!.'O3T\>7'Z[Y_>;U"8(MTT&PVG
MV6SE[M\]E6*P'TH+ID.V@W9]^OKHX\G'PQ<?7KZF#?[4!4I(%)(0U5CNAFS)
M6TA4Q1U0S/J<NE]PSV(K/)OXT=>R>E89L-2ENQM;8N=`%SR3IPBPN`AIZ!?(
MIZY89U)HF>N5OY\=;!([^/LC]'8KZ&3=.R!7:'?MMM/LY.]]_LON5MB=1.M?
M=O=@N^OM.RVW^3O:73Y&T_M`D$ZYDKFF#C,LGI:/WR&Z/Q\>OL>;A&@*4H`_
MBBD\0)SB7,%U6NV.?<->&1PJ]L<#7$X$>U%60)-)R.\/1$&=J9I01;Y*!5*J
M"`]O8HE8^7"8/%Q7V2%SDVV^.(I<A3;DK'>T\`MZ.<`!P(Y5QG(C2YRW0@`D
M"?T0Y]-#/BD+I@)U0U^)9]W/?ZAX4S[](TA7J,Q]SVGM[]NIG^(?3%0'!\.Z
M<2/SGJQDY@<\C%"4,D&!"L4CF4E%GZXF%<^NRDIPLN@:P]4&7'``D14U$*=V
M(*99VGFJV&_B7W"I(\FUE?$]#M>HN4TP701]V_,@:]70YPZT4[B@_O#8,7U2
MKDKZEKR.3:@(99!0S%[[DTD<E&5%6X0*R(!4:LP39^GQ:B>(&8HM*D53%OBI
M*LC0H'(*6O(K76V6,JDCHH\D$XQ'CR%!H99:/7S]A1WM\$UMQE9$+Y8M!?*]
M1NLMGFHA,3*<@);+U"A@OX%.7A@G'JR0=$L>UMKO;5DH@J7CM9Q.0T\9Z74<
MC`WC,.'^/!CCQ6J&ET&>F8NV<G,3^+;'KT&F%&?2F)H(-7==&-ZJB7A&1:#O
MEK8FW$<Z$V\XP`.C^(*V-)Z=)K,X%8]4/`!I)>&N:[>.8G&KFRHG7\/9J30W
M]540R+2)\<;+`:/+(=58KI3!`_EI(&^"(SJN4M@=T''^*3`HM)@&8-)R[9'&
M;'1+V3*!X@F2/</ZU`\CS;XH%&\9&,BW4*`&/A/47QB>O'-WY8)_B=Z9`(Z&
MAY7L36K:1J&3Q(&?X)L54?5LY(>3A)6',4^BW93-%LE8;:64(/TZI55PTE]9
M*=+1"PJD3D=HW&'J[A;E)](*#AC=4R%N8<*#AP3_MFO>HZ1569+'7%7#GS+9
MAB4ZOBY(V&:JMRDZC2:ITAI^'U65=T][5RA6+N2:=TFI#9%EH?^9M*J.U^9U
M9*#%;Y(#BB\2VD\?#U\?OGA?,2=T"^+03193&`Q:;:=CWFX@#I/FPTU)O$=1
M'*@UH0G?U5C.0<'^1&_#0BIE#'QE-:,@7`6S2KXZY`S0@-X2Q_HLJ;!^MC%N
M3T7P'S8FS1")C(+TO$DPSDF/-&@J2BKNXD07NA2)N3[93`$4M"K>\BI'8,5>
M7C@-,OVR`K8<;`A+O5G^5%C*W$,^J-P34D.E8B.K&I,=80J-1(0]K8:<\FTW
MKP(K2=)4$S$^Y)_KA#U]'&T5>D('8FG7S7I"QO5-[$@^J^!1][[LX?1#,*2?
MU@W?YZNBSHX).SNDBAUIUSLB\B@PEA.UC&VI+,$\/IMS_ZNR-:0B\%?@_8FY
M:!H*=?0)&6Z6$<36=B\TB_T1"/SC'TR^AB8[&"1RPJS!)P[HEWTWWTZ-1,Z<
M'=5/=T7FKI^B:7^QTY\Y'_7%Q$:>U=6A6K[-$'#_RJ_@<[E242Q3"5C<:<*C
M(<&D-BXZG7W0?COKNW\4[>?&R(<K_WI3RK<X6Z5[_+"(,L,;1B=!W[P*\2Y6
ML9SBF*<3=F"&S_8C6$M1[-AO-YS]CAE%*2.Y/KT(Q94=L?315V+FI/K;K@;,
M8D^^P4Q<[]G":R:2Q[);D2]*&W$^LZJJRRB'[UZI%[99-4JX;1U&"YY]=1K9
M*%Z-`QX'%NN%C`N\,IQ8'3*[0W"I_?8^@"+?5%"(6M=U]LT+7TLE8N1L_M4R
M(-12E/25(=%E)#4NK<CG\.5CZS*Z+7&?1CNVF(Y!,M*Q![PD,_D3=:H'K&&[
MIECJ$`,H5LB4+:=XB3+5%1G<EO998R&U&MI(H-]U;:[%;<D1K4(M%/`L?U$'
MX.P+=>QW`6W71+B;T;X_P-(>UV!<!-`8?(S@EEFP]0?/#ZB05`B<T9#!2LNQ
M#&<Q3D56VNVX3K?3R;S/Q:RFB6QNP[L>H!G<K:9A!3^8(4!X5K?=`I;<S)M<
M[L?2/?<QUC%8B&'/<[J]S.+VVAF_O-N=S/0KRX4*"].#?':PB2!O0#7I?;?;
M!B$RIQLV(H0U;DE1'DN*(M7T/,_I68O?C,D10!R$09/H9Y[3D34\#IL;SZ18
MKG8_?<+06O-7DY1!EC=/?T_D$II^(/,9>H&1VP96/<OL'X-5Z1+6!$O.57-.
M<2?V"Y%O=IQ>J[N,_-JT87W6L+4^:[!'"A.E[:PA5Z,D^<]'U$S*H!G?0-+0
M:[I.S[Q5<I.@X$D^O,^O_L\KZI7>FV&^4,-M,-B.^3\BT#5KO"N<A0W/:F;Z
MS68\)N'!$54-J(D84!M+D_1JQ"^SN4NR.G<1=<5RPJKL18ZWHNIREI)+4DIF
M`J)S"L;D_]]#XYI)5TCOK:;3:UOSK_M!E8?'4_#<$IK5L*R#1,M:RQFE!J:6
MB08W0U)D39X+\<]SO<SQ0#H=(N?R.HKI0?NIEN^SBXZZ<>T@>YB<%`VI0*^7
MV3%^?!GNGN;<4J)B/;6ZH*?V(QX47-H]O9EU7)2@&SWN_[7WK,UM&TE^UOX*
M,*FS*0E42/`I*]:5SG$NKOBQ9SN7ROE<+HB"))PI0DN`>G@W^]NO'_/H`084
M24FV4QM5)9:`F4%W3W=/STP_NCV`SEX:?(5D1EC]E.TC90>^&C/6\E6U;N1=
M>[X9[`?CG!0%^D^)@U1)RA#=,^A\GT\W2J\WS4&_D6.-B_:ML(>AE-GC_`@K
M<6C_8WRUL[%!?LRDKNEL!WU%-C:P.:Q725&.!-S9H'7*9CU^>E7,8AMLQ4&F
MA]?!P<[KG>!9/,U:/TX2V!<F,^K*<XYA7&VA7M>DF_*?J2-;W>O)9"725<AS
M&_S]?(19%-NC_E<EH>BBQ[,U0#UO(TF^0@E%6/V4'2%E=T?_BA+*DS?$R1OU
M_LBB)H(><%\K@A[X3T]:%RPA.31!#Z5F&/0PQ!J2;7]1A&YW-^S:L`<@]*\4
M?HEY-ZA.38:!)QBX,3W)'P5!DF+N#Y42-`Q4J#2&FJJL:ODI]J!I0;LM':L<
M:#@&O-G[BP[-Q"1U&.R4ZOP,?S&I_$I1>_`$/H#MFSJ3@:ZB,#MG_Q%;U^:3
MIGII$.,01TA-CCXDL]DT`]K3O^9:G^G1%M&'GXL>KRT]=H+@&94&8^8%''32
MA9W;DVI=ZGBY9]0+N[N.5?SC#Q_^Y^GK5\T'QUEV?$0Z[=L$R'5L/GR480`5
M;=U9!/0=1PDF[>A&NC+_!"VB_F!3G6%PSRW5E5U1^`,7Z0P#EJE$WV,53TA/
M`G80Y9.<HXPZLMOHA$Z19A_E)9>KO/':1+]1IVZJU%QWU`V[HUUI5'\&$D2;
M7P!='P?T>KMAS[H&\AXVP+1E#9B6[Q[B"9E(J`&*MH./\#OOQGFK\Q[;C5U7
M@L-X_!'==D6_37FM1N>@F/\`U#`I[&;ST]CF,$=?_V<OWSQ]374PJ1^.*Q)#
MX!(>84_.(/;[0U;<]'$)K#Q-XC6FUQL"NEWWKO)K0A?TPM_Y=.5ND/;.^6@8
M]FW58?P!NS`_!<W3-'P)?\1CJI0GN"R>Q+.S9GM3%Z?Y9'>?I4UNRXC%NT_O
MS7F$?%JYECY-XG,6%PN$SS)3B@ZHFF3'L)G49IST@H4_/:+PNZZ]0RUC/2+9
M)_B,0M%-^3>M0HTZM3>BE+Y-WRDKSAKU@:B./]G*1%V-#MLKX+(:M:JY%FG*
M)A,^E.&YW#:=E&6LK2AE$;]^>O#\^:LG%:MX>;+66,D1&HE#F]07_FKCHX&D
M/9N)^MR)),7UXW8.I;[Y(<G'L_0PP;PU8/REH(^O'P4?OC'N=N*F03D`V,'%
M3;F]=L"?RJ6#]^-,<VDXIOE8&H[TIZ>2UL@U'-UFG!2PUP$3LV;I#WOF,$!;
M%>1,.U9.7]LF[E-4#ARK^W%Q/5XN&&@G0&HOD^+Q`#3%7P]>'[SX^>EO;[2C
M5+EU1=?9*QE'G;JZ^;%/AVM53;D]K,.']#3O;&I;`!8'7[%1'TWN$DU2^I\1
M62]#1&`(V;P4RM_[&H/E\'?IYTUYFL:NUSL?Z^H8?,$S@``_785SR&C1Z;>K
M`1@F08>@:Z/)('`DNVGQ7M.>E*5[@6,:.3=:^G,-M>,LM7*UF;+#;+P.$NS0
M>:(N,Q&E5DM-T$><?"=?@!,-8<[.5XS)H)-WW<X)B4"P$@<L6\NOVX$]DRV3
MO6CF3?:'@(G]0$SXVK.GABI/W-X"XML,T0UXW]I7[+5MV6O;?GG;X'NW,R8G
MAZ#@8*J%(2L:6O6;Z<&3I9\MG#*O\/9'87?8]^GSL<X492A6BJO][>#ESYNH
M1QJ<W@(P?:^-!'NQ4BU50K*/>I$(J-_HF#V&OL4O[6F)8'+Q-;,N5MF='M=S
MO!W$Q`(IN`SOZSG6M.0Q>3Z;^`_==3T(FCH'2$O=LLN:"20G/2#RP+M`K$7D
M'<6U]T9H\2G%5`N(7,^S+L3KTWD[D!0._DW_6::U=\_2'X2]@2T-)<.R.?!Q
MC#&B1=N'I9:R&WBI]*AH+PB/-&6%>CT`K&]+0ZT-6(G^#]12_[X<9UNTW]\$
MEW_3UP[[-N*YALV4G6_#@Q31_*?S0"0PB5O!/&GMH_UAB9Q^2@C!)IG,^/X<
MK&-HKYNJ7RE,6)X(U4R/'B&DWSBE76A&$)T7]+6?Y$KAC'$E5M!^P-CF/#HG
M&.-3'OI-ZXU?7O[P2J@-(ZXT4FYVO6+S4+KBL*82]-NG[`,M6A+PL*<L'6:!
M)J^+67*29M/*.FTC\8H]8;1=A<%U>:W&Y7!?9SQ0BR-_U6GR/1V?TWI]I<Z?
M]D0/%GB&^<IRE64?C?+I6!V)7>M3+(KL]C)!52913*]MNW?794[56M&.K-\4
M]+V/"9%-'.!+>;RR4U0L%?9=A,&5F72'%%8RKQ;/('VX9O[$HK)`,BU6);ZJ
MWEF4`5&GS@TQQMA?9.MOQ=@XEUL()Q@\.1%@TIT7<?H>O-T+)EA<1%W;;$UL
MV7*UO\%145\J'.E80#*+XA7Z.%@YR";=8-L)<F=PMN!#`0^^)T'90RIB489-
MO4_@\@ZZ(H."P^E>?O:_GF>^=NJ9<LQO.&_M";W3O*4>E<[`U5%(-G&7FB&H
M\-Z@)C!]&17.Y!0ZN%->?O)*6@>IX-^'"W3]$JI^L:VQG))?W-=1\@ZT3)U;
MJ/OM0"AZL%[X+Z_&7U]C8U))JZ_=7`QKJFL<J#@[OT%I%UQ#G3G)Z#V/XF:C
MIJI?-0.A,[O'B)%3J)5T:>UV%30%]#@M5C:<:`QI*M5H:&D[K:RC[8S9Y(IF
M3FJ0<R^>72R-.%51Y.8"*W$%3=+G1:1>Q^-`"N@M%8NUO,J7&283*I^9DPOV
MEF;:\H(`A`(-_SUF<5]Q7<`A<9XB&![?;'5O6C&VY*S7KPT(L0(H1Y=#L4BP
M,^3G7R8XJZRG`_Y4UP6?J=_'=6*X*^]WV+>:_+(?!T9;,*>:^17^*@`5=*&0
M-N-MCP\Z=EEQ_>S!7MWBCYBWM_&[]QR0:\\,$S>0.>DJ^@-`VJU*N`;2&H=J
M@`%F:R$VO/,8@]5P]4[X;A0.VNTE8^@TCO?@!D]'U)Q'ZN7!B^=/7_(5I3C=
MP`XZ;I&H33P^)H6FO;CZHT[8']E@\#M`QT&C#O+&^/;P^Z:'LH9$N_;&:M"!
M^>I4$"S%8G@C,>;3^.PP/9EG\_R1=HW>*D=.E)/?-1[C&=23%S\\>?7+R[<J
MXQUAP*'T[!!&9SN?)@E\+L=3AAU,3"LNF1?$8PQZNS85`F'8'86#GLQ@N;%1
MNM?<V(`/D6A-DTM`^N.$LKEQZ2X'I4HROWI\>%;)0^S\^*H.)75)0I;154V1
MD$$?,!B*F':BU=45B#3-@`I<Q@_2W[COO=HS0%"$J*-.\,UY,CMS2$#WLEBA
MKDG$$-26D20F(X;1(TBK\6E&V7BE?@&K*BFH]&8VPS9VO`E='`.%^4.E@1O3
MY*J8`I-Q8CWZ34'D)(G@`Q:C6,/@/$D^4A?=W)*`(K)@97;RDF%$EVB#$JAA
MM^SOH&3X?U-EBH!Y&=A(II7GY<O,@P@U*:TQ<@WP+33ULV<"6O\H\^>5LR'H
MPE'7RIEQ.R@!O0RX:K7GF;?4K9@RYJEKSQCU8:]PGS][(]R0&NH-/CUX\1_/
M_O.75[^\(;<=YK5]LLNM?A$)*9_;A)2&0<6ZLK%1F6*79:WIHI#CU`M=4%+6
M8?8^B%?685^(1E3AL20F/*`1E)5)*'TNLHM$^ES0G\J9HO==9Q!$T:.H\Z@]
MD#48W6;L<]%YU.T\ZHV\O!ZU^V'4L;D..`490Y\5F&#EX>],91,[2,8+&@+P
MA@P`UU^P)?[<LT.VR#%1C_EW9\Q`C-GD,0T<5(M!`:EY:A4@V8_A=J#"(+^S
M!]P2`/NHW(TZ83<:EAP9+M)C,*%13:)6KKHTH`%_@4UH(XEN!,I>-!J4WDIJ
M%'$ZB8^.C)T""";G25Q@.QV4J8%Q[]9O`0P:/ZW.NB!YZ=7%;.,C+[W.9\G%
M'="KM0+!%#0^@JT%31W!EH3)2[%>-^Q:[RGOI&+VESOF,@6QEVH*HD5LMC)$
MR[):#5Q>RO5[87?07LAK=T"YRMPNH)R":!&_W1GEEH1+K$MJ*RN6)O/$>@0.
M<77J]V'9@=6I:U:G:DMV"NR.'O7]3H$1[.,BZPU.!D3@<6MO3M%59T('*>]A
M`3X_+R[-<1N>&6HG3AJ!EMSJ&%M3MC<NT^GE9NE@;JN`71ULZFQB;W*6R\?Y
M)H<3P2,52Y3F`49XA+#K5ZD.T\($J`?!](+3\F*6,S\:TPM"9#)E/Q1>!!4A
M9';(+T((]*.BJ+')Y,M1Q,LJHU$8[5I;AFX9LEQU-$99CMH'$2"^]Z$O*X4V
M5.+^,@,8FW,\A6U(<ZM`F4>D\_3L?,+EO,YG&>S"<@S,4.46*Q5D84.>\YFJ
M.BSB:#*B(QY_T\PKO+3Y<\=XX7PN:+IMBJ-^?NR%UID7Z207.D?]75.Q)S(:
MI]Q.E>P9/&H/_<L!!I]H%C(W'Y00^).\C=#Y8#[A[H%":>@DLB[J1N="XOL2
M'7RS]3CH:3\#&16KLJO3]1*)'19&0JV/96?+Q46U*]BN\!3^7(!'MP3<-P-]
MO"KPN0VFX@Y'84%WH+#YNK,Y,'HNY0NS(`WVZ78N2%LMV@/275Z*SA;%>Z5-
MWZ7OS5'^A+S/5&HG1&7@<\Z3J)BY4LC<,1P^$@_ZX=`)MM'.V>CMT\)^F^*2
M=SPOFFD88"5#G1),3$%3?=G`P"#QACD5GF<3'MDP3*M<G/Z6@/"'N50*GVWY
M(=LS@+E>N!4`?90;#L-1ST<Y4VKWGBDV'(6CH8]B*P#P62B%H>-=X]Q'`"&D
M(3NG%OROR=:N<*EQI+<E'*TO-L6MB>!$<U&;7<[2(@E4*"3U5X_P0J7BBV]=
M\<W7Y6N^7D('C<HKUVE?@E:IK$"BW;$]/BG_K7+/4.:&Q6(,FZ&6`7N0IL&=
MZHM]DVVE`B#?<UCD!+40KW*/=W1E9`.JZO'3NK2*`&LQ[42^"%ENMQ!E',([
M^4Y<JMJQE-SR*\Y(PA5:A7^0JPG]&AAOTF^^,>)BS[)+KM,_/WO^7`=Q:/][
MG'3S^[;7?UMQHW)>5FV9Q4C?(Y>57Q(FI8>!C6JI0T#C;.4*18TAMA4C_#5-
M=&S,=F#8EA:KC7HGR%3-6<.4#,FI>Z@`Y5<J8M`^,*#;>`+!Y'07ANV<IIKS
M^`%`(CPDU=@.Z-!7,Z`:8QN=3FN93O(*OC4?+C%<V15_-PH[]KCC=OJN(BH[
M.C37K6FZ>4=*4+O]+U"$.UI>_,IPQ\RI3V'L,`4K2M'V4.Y$OHZU<U7W,>7C
M7MR@-7W]B-.W%R)1I_E4((+5?JLHWAW:LWN5[P+*W*!`%].'+_0TE9A)*H!Z
M^S,O_N-QB1GI@]*!J(Z)'P?_+/=<3W\+MK4/*II<OU-LP7\P*"K&3BB>];6]
MB!WQZWDC076Z7LG0<OI^+5376A7$F-O&(+<19?DB_M-MG,5`28H9MK(X6&7B
MX)J74=S64EY>.9QN[H)1_KC&R0(K.F^+AHO0K(7)\&Q%LIS)\<J2MD`JK?T"
M=%.X4J<+!GEOX-U85S=<=[=AZ?1A8;3>$:M\][/L4SJP2>\,1G5T&>=W2`_7
MD[=AXB`\`0[F5,!;\Z_L;L[A0V7?6Z=&AY;X,990M'Y\CT5E>H)_G'/%`SN!
MF,!LY(TQK"/0G4_<0L)1G?J5R5<FFZEJH#V@)P[9FHJLFRM0S\=N47LH+]QI
M+:CZDBE'?NM&A@[_["]6B@P(%/`J,L"L9SK_AN\@3)?[4TUXM>#J?QV8M&Z$
M16>Z4?`HX&>UG!?KM/'>!'FTBMK2D8:AHJ@71MU('F)\'210)X*>&)3%&(>!
M8!9J,@ED"(2/$%[>B-IAU'-XHYQ4"XO611A#A_]%YAJGF,T!=R0!%M/:FB="
M)HFVN4(OY"L@A0:&(E'12#IRQPN.',L!%*KZL.;]!K=J;!6>\/GS8W5SH^\5
M[#<C'^75\ND+.PS).BHB`:&G92S2[FP$\)']@,`KX)>"<E?DT3N5CJ*@WP"M
MJ-4"O.#_EI>/@TKP8QY9=8!L0RR'U51LY(T,N'D@`G$Z-O#+!/X\!N+(AQ1Z
M!!]U'C+K`>"MH)#/3?E9&QJB]]].@%`IS$#')ED.+:H!2TI-*XG?#X08.+EX
M8IE[<D5Q4GF8`Z78[U2!=$=A-'#.C>]93NA@P]Y[,KM];]YO*Z&_693HK3/*
MX\?E8>Y.R##S45F<^!MW(3P:C1:TN$ELA)R8.+7[%QBEO6\M+V"_V[7AGN2E
MM/RX<O-YEZ$>BE=EIU`-`#82PV%6F'%4E*8"27@$S+6OXJPHK$UM-Y6[@,D+
M6YC"S^8`2\45.8E&Q'.]\JM$J-]M!?\U3V;7=,T\SY-9&&`&6-4X#O`6&C0>
MHA"/BV2&N5_/P6S$O)AOH<O?YDE.=Y(IYFR-\WQ^!I-59,%I?)$$APG,#^7J
MA(?Q!'-A7O,'5`E+Y>(P28[!)CZ#KFE<))/K(#Z&;\&`V$Q_,4C&IQFFW0R:
M+]*3TR(8QP`PP'@^RPXGR1FJ"2Q$'A3Q1ZR(6039%`#!(?""'T;#N=[9#(+O
MW\:'^_A=!&P&`D3@Y<'#ZX=[`-UUD%$B42P5-<LF`GFW"R$</)P^Q!RA^&U,
M3AKC?S"$Z93K2_Q#2K83,HP*S"F.$`-\)S&FDH,1-;9,IFDF!\*QD8;PO2?H
M0@F#'&>3(T\J4H"2%?X`C.;=;GD+LA)'MBAM(H9>K\R!VW0V9L81M3^Q2`%Y
M*Q2S3?('5@ZBNG0`;F,Z5(YM'Z/YN-SHNU9$^O4%[#?8C5@%WIE>VZH7+R:J
MW>>0`XS)#+:D.*PH#/"CQKA)+&J$@G_4&&M)B)8/-<9MI$2/4>ZZK*Q(7):0
MFZK4E.BQG!#5_'AD2_C77*3"N0;_\*2%'FYT=OK&L\9IQ&Y\4;<NM]^H&XZ,
M=ZHY?UQ<C47$N2@&KAQA<GRC*-?T;FQ<(&0@"%H7G(H.H6=@ROG85P6&/2=O
M!XWW0*K=#CN1+YL]#G!3^.8BH@75:"$9ORCTXD)2D*)U`R[EIVPI-E/=3D3>
ME#^H3YZUX8UA\OK>6O\M',3<.,H%(:$E(,507.:H5"EI';2"IB^BR$-X]??8
M/5:C:>Y5V'#=:1;L>._SK)R&UYOH^&*=J:Z&R\HIC2]6GM0Z'&Z:54OGRK1Z
MI7G0"SO#;BG)54EY*.L%5N^WN#H<PI(&VY6C28+YVM$-&W<.JJH!4`!+-,['
MI[AX'*&^/PBF<W*JS8YY$Z&W+6*5H=46MT!JI=>%UVF'0JOC>#+GY8,7-EAA
M3F#AQU->/1Y[[EW@YAC>TY>"DRPXRBZGE_'L*.2E]#(%.@+IX'^P4,7J>V:T
M$QA`#TA+M@OZCILA^&.$4X'J5'.^FX^\LV=,.:IAJ#1\9]`%JD>EO#3W1'5>
MG6F)-6BL27LQ4G465J&_'.F&N?#/@OVY[7SX3\'[812UEY6*E]EE<(G6'=NA
MR15Z920$.,:[$2YJID)LD"=)<$FV&<PI&$QSV&Y?J^%.,D)/1H)QR^E1.H8Y
MSP-E2YJ1376:XC0N<$0LP#`YTD(Q05]7L1&$-O@7)B#)CNW<A_`/69HPRE1!
M9WOI@SD<V$",DPNDA"]>8FN%-AJ&%"M\3=,)?'%&=G`,.QTT,9,IPI/IDYKL
MS%1$H62?!70'HDPFT/M'`#>YBC'[;XA?RI-X-CY%CVW82%TBW^.&*M872]KZ
M#/&$!S!A.()Q.AO/S_(BGHX5-RE*3/";0<:VL.F]HX;#C\>:-H9,P=D\+[1$
MN!U!_C(Y/"P'N9R$([&Q$&;\L3/70*X\*8@!6E1QMW4(#3\F!7&744RT845P
M<+"+>#)/<#+E0$P")J\D0!B<)?$4:8CLHN?5,J\8=PNZS`IHNF7+SUAQH^L0
MFVG5#5TT@G*9S8[4L\MC'2[9`^'J+*O\;B5<0LT(KEU5PAQU52-NRPB:,\X2
M4E<K;W*<-62/V2)PX5E3"&='I7%()-<51KE4E>5R>9%T\5HDGS=(IKM,+2VE
M)3ERX;F%M);F?6F9K?[<3HK]EX-X:^H4QA&]S%XC.R_R=^AZN/GR%0:NHWD\
MS=`$WA/+^)@.EHJV,LQ!"QQ,L0P"S%HVAND#M$&1C%$ZI@\+$E1E.Y15`C`^
M-B$/'O@(,'?"QS5`/BRO]?+5!P2#N!;I=S0'@43I5B*KAE%Z3^:U<4DE7<)(
ME?%-<K]\D_P5T828:0,E]E:$"58FC?]0OQ-&_;;(5AR4,NS)->'7F,[5N'94
M*YNE"1WW(:+_#AN$*9]B80!\"/JW2&?:',5C*;)[F31Z.4UF9^DT1K%1J742
M<S"(M:Q<.PEV]6!F\_$@:C1%5-1":KPC?'Z9%F",GY_/LO,9'RRB45XP:.<9
M^J<`J?\/=9>68U36]DO:QVJ*]0_B/'<P*>U#=,:>CY'UQ47?0/2^>O'JOY_J
M/3569D@NZ98%0V</LXF:,LYHJ(J/1[T()J,C,C3?]V0(O88<>ZL)\2YKO$BL
M-3'.<)59JIV?^I][FCF?6'6C*.Q&MH3I@JSI9CZQAIO=UNFC:\Y5$Q].--[`
M)REM^\A4T*H(C8OS9)P>Z_U,')S,<*^*1]ASC`Q%0\?=TL47&-U]*`_^<Y=(
M5!YAGQ!/$''.HXE8J%I=[AN-%"!SQ-!3=B$9C8BE.[HFAGTINOSUEG21;*1H
MM"YU=NZ1/%XV&K3#[F#X+\]&^?PP+])B[K#2H`.T&?W)2GX2^=BIU^^'O4'Y
MT+%\70%C@HE.]:5LC=330*:-$`X0C?&IQS%E/E5-X:WUL@$+^2)EH[EIG1G$
M\2WECJ;Z"?W2-G49(.FNQ@5TTU[--KPLXCJ8K@FVC]:#;B\<]*SH7J0VT]I-
M;,K,)QGO,-';VL/D))W2SD0MQNSB03'!4QUHK0^G\0=Z\Y<#6"P]8W`7Q366
M(9O:F;4AG5FITA4Z'*G".1<I=*?AJ&0(V\IFG;RY9,@?0&L1NA8E(9+NFYL7
MOT%W`"QA;AA78HEG:[.$4%F2.Y;G"[/5^$*L\=4K[;OA$*\2P:QYN]7T32`M
MAPGF)E^OX)E/L;4ZF](A451"6V::W%I:#=O,OKZC2FA.S;-&^9[!%Q3LAG)5
M^]AH3KGMEX$Y3(3R'J"Q5"4T&VZZ;#4T=8&)K\8V[(_TA4J9.3V<8,DSZ_PC
M<BE6*H+IO,(^4(D-/(7;F#V6`Y<&EP7<3/%S9<!P`XQ_X$20?6#I<@FW12Q]
M<R6WA9R\-%OB^\.Z`F]",@Y%&;=E9,/EMD:#*[#51.ZNS(*^TFZ6YU8O[V:Y
M3S^_.QZL+T.';U>'U;+>H2Q'5\^`/AT[;/?"82_RZ5@ZG?Y3Q7X!%9NLH6*M
MNT@I7'HI3;8L;ZLCX8:`4S`9!_E]_4IX=Q0.HVKJREJ>_U,'+^+1!3JXEBE7
MU':?A3V_3OT\ZH9#ZR2JDHP;'Z]B/LDN@6/U(WTX:\-MI0+E70B0Q1_3.ASL
MAL.A/7NX[;=X2U3W12^VNX#M[LC6)Q@.`:1JF*WE>"5\HD3X^2S)\>Z9@*5L
MQ'A,@O>5V;1(I_/DF]L5"U?LTTAS)H=PL?0"Q,[H/FQ'G7XXBLSEY89)3V+N
MZ'0%"S=Q"8NT30Q19`S)^!1SO#33H_0D+?`OC%AM/>P\Q)QT@^`1_!X_-/07
M65I&[2@<M>UAXMJ`S&L`\<"Q60;$2Z$A`.96R,FSPYW\Y(/.W]!T_OY'\/K@
M5PQ!_.?3)S^],IHL&Q>3YIN?WK[]+0S>/GOUY,W3MR_#X`%T)2B^3:9'Z;%V
M2JGWGK5<]VUZ#.KE./CIX,T'&%"#EA1XA=24@Q)Y^[OA:.`F*;UG+$J.H:M`
*_/\_+SCW6_L`````
`
end

-----BEGIN PGP SIGNATURE-----
Version: 2.6.2

iQCVAwUBMZMAsXD/+HJTpU/hAQEoCQP/eBuRLFbOf64KZYXQD1fmqmN6Y22xu7+y
PmB8SSpdTTLULtPe66Lddz52up55B1WIHvECAB0sipr3KdPzHIC3FHctxlkLaQ7K
a74swlrWGjfH0yttzIA3rhM6OfViHLYZvZ7+InPUhSQ2IepC4zuDtiJvyE/X8nV4
LyqTBdno5nQ=
=EIDN
-----END PGP SIGNATURE-----

Re: ZLE 8-bit patch

[Wayne Davison]

Zefram writes:
> The i-search code is now a real mess.
> Someone should review/rewrite the code properly.

I'm fiddling with it now.  If anyone else is working on it, let me know.

Also, I discovered that your DEBUG code in zstrlen() isn't checking
the error condition correctly.  Here's a patch:

Index: utils.c
@@ -2936,11 +2936,12 @@
 
     for (l = 0; *s; l++)
 	if (*s++ == Meta) {
-	    s++;
 #ifdef DEBUG
 	    if (! *s)
 		fprintf(stderr, "BUG: unexpected end of string in ztrlen()\n");
+	    else
 #endif
+	    s++;
 	}
     return l;
 }

..wayne..

Updated isearch patch based on ZLE 8-bit patch

[Wayne Davison]

I've updated the incremental search code based on Zefram's latest 8-bit
patch.  I am still using the beta 17 release with only my isearch and
Zefram's 8-bit ZLE patches applied, but since the 8-bit patch applied
with minimal difficulty I think that this diff should also apply easily
to the code with the other published patches.

This patch includes my small fix to ztrlen(), so if you already applied
that it will probably be easier to unapply it before running this through
patch.

I fixed lots of glitches in the new interaction between a meta-expanded
line and the non-meta-expanded lines.  I also added a "Failing" string
to the mini-buffer prompt when the search fails and fixed the way the
isearch code handles backing up through failure cases (it no longer rings
the bell if backing up to a previous failure, for instance).

Discussion of internals:

I added two meta-related functions to utils.c since I thought they might
be useful.  I only use each one once, however, so if you don't want the
new functions, integrate them into zle_hist.c.  They are:

1. metalen(str, len) -- this function takes a pointer to a metafied
string and a length from the PRE-metified version of the string and
returns the length including any meta characters.  I use this function
to convert "cs" into a "pos" in the metafied line.

2. ztrsub(s2, s1) -- this function performs pointer subtraction in a
metafied string, returning (s2 - s1) - the_number_of_meta_chars_in_
_the_substring.

When doisearch() converts the current search string into its metafied
form, I decided to just use the META_STATIC return and limit the search
string to PATH_MAX characters.  If you don't like this, META_USEHEAP
could be used instead -- I just didn't like the thought of a new alloc
for each typed character (even when backing up).

I needed to add extra variables to the isearch_spots array, so I decided
to save some memory by making some of the ints unsigned shorts.  If you
don't like this, change them back to ints.

I modified hstrnstr() to have a separate forward and backward loop,
making the code more efficient.

Finally, I eliminated one extra call to refresh() that didn't seem to be
needed.

I tested this with various searches, including some lines with null
characters on them, and it worked fine.

..wayne..
---8<------8<------8<------8<---cut here--->8------>8------>8------>8---
Index: utils.c
@@ -2872,6 +2872,24 @@
     return s;
 }
 
+/* Return the character length of a metafied substring, given the      *
+ * unmetafied substring length.                                        */
+
+/**/
+int
+metalen(char *s, int len)
+{
+    int mlen = len;
+
+    while (len--) {
+	if (*s++ == Meta) {
+	    mlen++;
+	    s++;
+	}
+    }
+    return mlen;
+}
+
 /* This function converts a zsh internal string to a form which can be *
  * passed to a system call as a filename.  The result is stored in a   *
  * single static area.  NULL returned if the result is longer than     *
@@ -2926,7 +2944,7 @@
 	return 1;
 }
 
-/* Unmetafy and check the length of a string. */
+/* Return the unmetafied length of a metafied string. */
 
 /**/
 int
@@ -2936,11 +2954,33 @@
 
     for (l = 0; *s; l++)
 	if (*s++ == Meta) {
-	    s++;
 #ifdef DEBUG
 	    if (! *s)
 		fprintf(stderr, "BUG: unexpected end of string in ztrlen()\n");
+	    else
 #endif
+	    s++;
+	}
+    return l;
+}
+
+/* Subtract two pointers in a metafied string. */
+
+/**/
+int
+ztrsub(char const *t, char const *s)
+{
+    int l = t - s;
+
+    while (s != t)
+	if (*s++ == Meta) {
+#ifdef DEBUG
+	    if (! *s || s == t)
+		fprintf(stderr, "BUG: substring ends in the middle of a metachar in ztrsub()\n");
+	    else
+#endif
+	    s++;
+	    l--;
 	}
     return l;
 }
Index: zle_hist.c
@@ -572,17 +572,20 @@
 extern int ungetok;
 
 struct isrch_spot {
-    int hl;		/* This spot's histline */
-    int pos;		/* The search position on the line */
-    int len;		/* How long the search string was */
-    int dir;		/* The direction we're searching */
+    int hl;			/* This spot's histline */
+    unsigned short pos;		/* The search position in our metafied str */
+    unsigned short cs;		/* The visible search position to the user */
+    unsigned short len;		/* The search string's length */
+    unsigned short flags;	/* This spot's flags */
+#define ISS_FAILING	1
+#define ISS_FORWARD	2
 } *isrch_spots;
 
 static int max_spot = 0;
 
 /**/
 void
-set_isrch_spot(int num, int hl, int pos, int len, int dir)
+set_isrch_spot(int num, int hl, int pos, int cs, int len, int dir, int nomatch)
 {
     if (num >= max_spot) {
 	if (!isrch_spots) {
@@ -595,98 +598,121 @@
     }
 
     isrch_spots[num].hl = hl;
-    isrch_spots[num].pos = pos;
-    isrch_spots[num].len = len;
-    isrch_spots[num].dir = dir;
+    isrch_spots[num].pos = (unsigned short)pos;
+    isrch_spots[num].cs = (unsigned short)cs;
+    isrch_spots[num].len = (unsigned short)len;
+    isrch_spots[num].flags = (dir > 0? ISS_FORWARD : 0)
+			   + (nomatch? ISS_FAILING : 0);
 }
 
 /**/
 void
-get_isrch_spot(int num, int *hlp, int *posp, int *lenp, int *dirp)
+get_isrch_spot(int num, int *hlp, int *posp, int *csp, int *lenp, int *dirp, int *nomatch)
 {
     *hlp = isrch_spots[num].hl;
-    *posp = isrch_spots[num].pos;
-    *lenp = isrch_spots[num].len;
-    *dirp = isrch_spots[num].dir;
+    *posp = (int)isrch_spots[num].pos;
+    *csp = (int)isrch_spots[num].cs;
+    *lenp = (int)isrch_spots[num].len;
+    *dirp = (isrch_spots[num].flags & ISS_FORWARD)? 1 : -1;
+    *nomatch = (isrch_spots[num].flags & ISS_FAILING);
 }
 
+#define ISEARCH_PROMPT		"failing XXX-i-search: "
+#define NORM_PROMPT_POS		8
+#define FIRST_SEARCH_CHAR	(NORM_PROMPT_POS + 14)
+
 /**/
 void
 doisearch(int dir)
 {
-    char *s, *ibuf = halloc(80), *sbuf = ibuf + 14;
-    int sbptr = 0, cmd, top_spot = 0, pos = cs, sibuf = 80;
+    char *s, *ibuf = halloc(80), *sbuf = ibuf + FIRST_SEARCH_CHAR;
+    int sbptr = 0, cmd, top_spot = 0, pos, sibuf = 80;
     int nomatch = 0, skip_line = 0, skip_pos = 0;
     int odir = dir, *obindtab = bindtab;
 
-    strcpy(ibuf, (dir == -1) ? "bck-i-search: " : "fwd-i-search: ");
-    statusline = ibuf;
+    strcpy(ibuf, ISEARCH_PROMPT);
+    memcpy(ibuf + NORM_PROMPT_POS, (dir == 1) ? "fwd" : "bck", 3);
     if (histline == curhist) {
 	zsfree(curhistline);
-	curhistline = metafy(UTOSCP(line), ll, META_DUP);
+	s = curhistline = metafy(UTOSCP(line), ll, META_DUP);
     }
+    else
+	s = metafy(UTOSCP(line), ll, META_USEHEAP);
     bindtab = mainbindtab;
-    s = curhistline;
+    pos = metalen(s, cs);
     for (;;) {
 	/* Remember the current values in case search fails (doesn't push). */
-	set_isrch_spot(top_spot, histline, pos, sbptr, dir);
-	if (sbptr == 1 && sbuf[0] == '^')
+	set_isrch_spot(top_spot, histline, pos, cs, sbptr, dir, nomatch);
+	if (sbptr == 1 && sbuf[0] == '^') {
 	    cs = 0;
-	else if (sbptr > 0) {
-	    statusline = ibuf = metafy(ibuf, sbptr + 14, META_USEHEAP);
-	    sbuf = ibuf + 14;
+    	    nomatch = 0;
+	    statusline = ibuf + NORM_PROMPT_POS;
+	} else if (sbptr > 0) {
+	    char *mbuf = metafy(sbuf, sbptr, META_STATIC);
+	    char *last_line = s;
+
 	    for (;;) {
 		char *t;
 
 		if (skip_pos) {
-		    pos += dir;
-		    if (pos < 0 || sbuf[0] == '^' || pos >= strlen(s))
+		    if (dir < 0) {
+			if (pos == 0)
+			    skip_line = 1;
+			else
+			    pos -= 1 + (pos != 1 && s[pos-2] == Meta);
+		    } else if (sbuf[0] != '^') {
+			if (pos >= strlen(s+1))
+			    skip_line = 1;
+			else
+			    pos += 1 + (s[pos] == Meta);
+		    } else
 			skip_line = 1;
 		    skip_pos = 0;
 		}
 		if (!skip_line && ((sbuf[0] == '^') ?
-		    (t = (strncmp(s, sbuf + 1, strlen(sbuf + 1))) ? NULL : s) :
-		    (t = hstrnstr(s, pos,  sbuf, strlen(sbuf), dir)))) {
-		    char savet = *t;
+		    (t = (strncmp(s, mbuf + 1, strlen(mbuf + 1))) ? NULL : s) :
+		    (t = hstrnstr(s, pos, mbuf, strlen(mbuf), dir)))) {
 		    setline(s);
-		    *t = 0;
-		    pos = strlen(s);
-		    cs = ztrlen(s);
-		    *t = savet;
-		    if (dir == 1)
-			cs += sbptr - (sbuf[0] == '^');
+		    pos = t - s;
+		    cs = ztrsub(t, s) + (dir == 1? sbptr - (sbuf[0]=='^') : 0);
 	    	    nomatch = 0;
+		    statusline = ibuf + NORM_PROMPT_POS;
 		    break;
 		}
 		histline += dir;
 		if (!(s = qgetevent(histline))) {
+		    if (sbptr == (int)isrch_spots[top_spot-1].len
+		     && (isrch_spots[top_spot-1].flags & ISS_FAILING))
+			top_spot--;
+		    get_isrch_spot(top_spot, &histline, &pos, &cs, &sbptr,
+				   &dir, &nomatch);
 		    if (!nomatch) {
 			feep();
 			nomatch = 1;
 		    }
-		    if (sbptr == isrch_spots[top_spot-1].len)
-			top_spot--;
-		    get_isrch_spot(top_spot, &histline, &pos, &sbptr, &dir);
-		    s = curhistline;
+		    s = last_line;
 		    skip_line = 0;
+		    statusline = ibuf;
 		    break;
 		}
-		pos = dir == 1? 0 : ztrlen(s) - sbptr + (sbuf[0] == '^');
-		skip_line = (pos < 0 || strcmp(curhistline, s) == 0);
+		pos = dir == 1? 0 : strlen(s);
+		skip_line = !strcmp(last_line, s);
 	    }
-	    unmetafy(ibuf, NULL);
-	} else
+	} else {
 	    top_spot = 0;
+    	    nomatch = 0;
+	    statusline = ibuf + NORM_PROMPT_POS;
+	}
 	sbuf[sbptr] = '_';
-	statusline = ibuf;
-	statusll = sbptr + 15;
+	statusll = sbuf - statusline + sbptr + 1;
     ref:
 	refresh();
 	if ((cmd = getkeycmd()) < 0 || cmd == z_sendbreak) {
-	    get_isrch_spot(0, &histline, &pos, &sbptr, &dir);
+	    int i;
+	    get_isrch_spot(0, &histline, &pos, &i, &sbptr, &dir, &nomatch);
 	    s = qgetevent(histline);
 	    setline(s);
-	    cs = pos;
+	    cs = i;
 	    break;
 	}
 	switch (cmd) {
@@ -702,15 +728,21 @@
 	case z_vibackwarddeletechar:
 	case z_backwarddeletechar:
 	    if (top_spot)
-		get_isrch_spot(--top_spot, &histline, &pos, &sbptr, &dir);
+		get_isrch_spot(--top_spot, &histline, &pos, &cs, &sbptr,
+			       &dir, &nomatch);
 	    else
 		feep();
+	    if (nomatch) {
+		statusline = ibuf;
+		skip_pos = 1;
+	    }
 	    s = qgetevent(histline);
 	    if (!sbptr || (sbptr == 1 && sbuf[0] == '^')) {
+		int i = cs;
 		setline(s);
-		cs = pos;
+		cs = i;
 	    }
-	    memcpy(ibuf, (dir == 1) ? "fwd" : "bck", 3);
+	    memcpy(ibuf + NORM_PROMPT_POS, (dir == 1) ? "fwd" : "bck", 3);
 	    continue;
 	case z_acceptandhold:
 	    acceptandhold();
@@ -725,31 +757,30 @@
 	    acceptline();
 	    goto brk;
 	case z_historyincrementalsearchbackward:
-	    set_isrch_spot(top_spot++, histline, pos, sbptr, dir);
+	    set_isrch_spot(top_spot++, histline, pos, cs, sbptr, dir, nomatch);
 	    if (dir != -1)
 		dir = -1;
 	    else
 		skip_pos = 1;
 	    goto rpt;
 	case z_historyincrementalsearchforward:
-	    set_isrch_spot(top_spot++, histline, pos, sbptr, dir);
+	    set_isrch_spot(top_spot++, histline, pos, cs, sbptr, dir, nomatch);
 	    if (dir != 1)
 		dir = 1;
 	    else
 		skip_pos = 1;
 	    goto rpt;
 	case z_virevrepeatsearch:
-	    set_isrch_spot(top_spot++, histline, pos, sbptr, dir);
+	    set_isrch_spot(top_spot++, histline, pos, cs, sbptr, dir, nomatch);
 	    dir = -odir;
 	    skip_pos = 1;
 	    goto rpt;
 	case z_virepeatsearch:
-	    set_isrch_spot(top_spot++, histline, pos, sbptr, dir);
+	    set_isrch_spot(top_spot++, histline, pos, cs, sbptr, dir, nomatch);
 	    dir = odir;
 	    skip_pos = 1;
 	rpt:
-	    memcpy(ibuf, (dir == 1) ? "fwd" : "bck", 3);
-	    refresh();
+	    memcpy(ibuf + NORM_PROMPT_POS, (dir == 1) ? "fwd" : "bck", 3);
 	    continue;
 	case z_sendstring:
 	    sendstring();
@@ -778,10 +809,14 @@
 		goto brk;
 	    }
 	ins:
-	    set_isrch_spot(top_spot++, histline, pos, sbptr, dir);
-	    if(sbptr == sibuf - 16) {
-		statusline = ibuf = hrealloc(ibuf, sibuf, sibuf * 2);
-		sbuf = ibuf + 14;
+	    if (sbptr == PATH_MAX) {
+		feep();
+		break;
+	    }
+	    set_isrch_spot(top_spot++, histline, pos, cs, sbptr, dir, nomatch);
+	    if (sbptr == sibuf - FIRST_SEARCH_CHAR - 2) {
+		ibuf = hrealloc(ibuf, sibuf, sibuf * 2);
+		sbuf = ibuf + FIRST_SEARCH_CHAR;
 		sibuf *= 2;
 	    }
 	    sbuf[sbptr++] = c;
Index: zle_utils.c
@@ -265,17 +265,22 @@
 char *
 hstrnstr(char *str, int pos, char *t, int len, int dir)
 {
-    char *s;
+    char *s = str + pos;
 
-    for (s = str + pos; dir == -1 || *s; ) {
-	if (!strncmp(t, s, len))
-	    return s;
-	if(dir == -1) {
-	    if(s == str)
-		break;
-	    s -= 1 + (s>str+1 && s[-2] == Meta);
-	} else
+    if (dir > 0) {
+	while (*s) {
+	    if (!strncmp(t, s, len))
+		return s;
 	    s += 1 + (*s == Meta);
+	}
+    } else {
+	for (;;) {
+	    if (!strncmp(t, s, len))
+		return s;
+	    if (s == str)
+		break;
+	    s -= 1 + (s != str+1 && s[-2] == Meta);
+	}
     }
     return NULL;
 }
---8<------8<------8<------8<---cut here--->8------>8------>8------>8---