mirror/emacs
1
0
Fork 0
mirror of https://git.savannah.gnu.org/git/emacs.git synced 2024-12-24 10:38:38 +00:00
Code Issues Releases Activity

* src/sysdep.c (child_setup_tty): Comment-out left-over non-ICANON code.

Browse Source
This commit is contained in:
Stefan Monnier 2010-08-04 20:10:07 +02:00
parent e0d62a9b7b
commit aaa0a19a94
2 changed files with 20 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/ChangeLog
View File

@ -1,3 +1,7 @@
2010-08-04 Stefan Monnier <monnier@iro.umontreal.ca>
* sysdep.c (child_setup_tty): Comment-out left-over non-ICANON code.
2010-08-03 Johan Bockgård <bojohan@gnu.org>
* data.c (Flocal_variable_p): Handle variable aliases correctly.

29
src/sysdep.c
View File

@ -503,26 +503,29 @@ child_setup_tty (int out)
s.main.c_cflag = (s.main.c_cflag & ~CBAUD) | B9600; /* baud rate sanity */
#endif /* AIX */
/* We used to enable ICANON (and set VEOF to 04), but this leads to
problems where process.c wants to send EOFs every once in a while
to force the output, which leads to weird effects when the
/* We originally enabled ICANON (and set VEOF to 04), and then had
proces.c send additional EOF chars to flush the output when faced
with long lines, but this leads to weird effects when the
subprocess has disabled ICANON and ends up seeing those spurious
extra EOFs. So we don't send EOFs any more in
process.c:send_process, and instead we disable ICANON by default,
so if a subsprocess sets up ICANON, it's his problem (or the Elisp
package that talks to it) to deal with lines that are too long. */
/* There is no more "send eof to flush" going on (which is wrong and
unportable in itself), and disabling ICANON breaks a lot of stuff
and shows literal ^D in many cases. The correct way to handle too
much output is to buffer what could not be written and then write it
again when select returns ok for writing. This has it own set of
problems. Write is now asynchronous, is that a problem?
How much do we buffer, and what do we do when that limit is reached? */
process.c:send_process. First we tried to disable ICANON by
default, so if a subsprocess sets up ICANON, it's his problem (or
the Elisp package that talks to it) to deal with lines that are
too long. But this disables some features, such as the ability
to send EOF signals. So we re-enabled ICANON but there is no
more "send eof to flush" going on (which is wrong and unportable
in itself). The correct way to handle too much output is to
buffer what could not be written and then write it again when
select returns ok for writing. This has it own set of
problems. Write is now asynchronous, is that a problem? How much
do we buffer, and what do we do when that limit is reached? */
s.main.c_lflag |= ICANON; /* Enable line editing and eof processing */
s.main.c_cc[VEOF] = 'D'&037; /* Control-D */
#ifdef 0 /* These settins only apply to non-ICANON mode. */
s.main.c_cc[VMIN] = 1;
s.main.c_cc[VTIME] = 0;
#endif
#else /* not HAVE_TERMIO */