zsh-workers
 help / color / mirror / code / Atom feed
* zsh-2.6beta12 bug
@ 1995-11-29 17:21 Heiko Schroeder
  0 siblings, 0 replies; only message in thread
From: Heiko Schroeder @ 1995-11-29 17:21 UTC (permalink / raw)
  To: zsh-workers


Hi!

I think I found a small bug in zsh-2.6beta12: the shell won't correctly
process input lines with 8 bit (Latin-1) characters. Processing seems to
stop at the character with it and the rest of the line being discarded.

The cause seems to be line 130 of input.c. The line

	    return lastc = (unsigned)*inbufptr++;

near the beginning of ingetc() should read:

	    return lastc = (unsigned char)*inbufptr++;

The original line may still produce a negative value for characters which
have bit 7 set. (At least with gcc 2.7.0 on a i486 running linux, I
couldn't test it under different environments.)

Perhaps it would be even cleaner to change the declaration of inbufptr
to "unsigned char *" instead of "char *".

CU
   Heiko

PS: I'm not subscribed to this maling list.

--
+----------------------------------------+------------------------------------+
|Email:                                  | Snail-mail:  Heiko Schroeder       |
|                                        |              Lerchenweg 120        |
|  heiko@pool.informatik.rwth-aachen.de  |              52223 Stolberg        |
|                                        |              GERMANY               |
+----------------------------------------+------------------------------------+


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~1995-11-29 18:15 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
1995-11-29 17:21 zsh-2.6beta12 bug Heiko Schroeder

Code repositories for project(s) associated with this public inbox

	https://git.vuxu.org/mirror/zsh/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).