Ctrl-Z does not work in Windows

[lifthrasiir]

fn main() {
    for l in std::io::stdin().lock().lines() { println!("{}", l.map(|s| s.escape_unicode())); }
}

As of rustc 0.13.0-nightly (126db549b 2014-12-15 00:07:35 +0000) this does not work: pressing Ctrl-Z then Return (a standard EOF sequence in Windows) simply prints Ok(\x1a\x0d\x0a), and pressing Ctrl-C immediately terminates the process while printing a panic! message.

This is apparently a regression of #3948. According to the relevant code from CPython, ReadConsoleW may return an error ERROR_OPERATION_ABORTED in both cases, and msvcrt (I think) will additionally issue a SIGINT for Ctrl-C within milliseconds. The current code handles none of them.

[blindley]

I ran into this issue as well. Although lifthrasiir's code does not compile for me in the current version (rustc 1.0.0-nightly (f4f10dba2 2015-01-17 20:31:08 +0000)), I used this code instead.

fn main() {
    for line in std::io::stdin().lock().lines() {
        match line {
            Ok(line) => println!("{:?}", line.as_bytes()),
            Err(e) => panic!("{}", e),
        }
    }
}

[steveklabnik]

Triage: i do not have access to Windows to test this, but here's updated code that compiles:

use std::io::BufRead;

fn main() {
    let stdin = std::io::stdin();
    for line in stdin.lock().lines() {
        match line {
            Ok(line) => println!("{:?}", line.as_bytes()),
            Err(e) => panic!("{}", e),
        }
    }
}

[shepmaster]

i do not have access to Windows

I use a modern.ie VM.

c:\rust\ctrlzed>cargo run
     Running `target\debug\ctrlzed.exe`
^Z
[26]
^C
c:\rust\ctrlzed>

That is, control-Z followed by enter does not terminate the program, but prints out [26]. Control-C terminates the program without a panic.

[alexcrichton]

This should be a pretty easy bug to knock out, so marking as E-easy and E-mentor as I'm willing to mentor it.

This'll involve poking around src/libstd/sys/windows/stdio.rs and looking specifically at the call to ReadConsole. The usage of try! there will likely need to be eschewed in favor of manually checking the Result returned from cvt.

I'm unfortunately not sure of a great way to add an automated test for this, but manual testing should work just fine!

[Xinayder]

Can I pick this up?

[alexcrichton]

@RockyTV sure! Just let me know if you need any help!

[Xinayder]

@alexcrichton I started working on this today, and I'm not sure what to do. I'd like to know what should I do if I remove the try! call: should I use a match, and if so, what do I do with the value of Some? Do I assign it to a variable, or do I call another function?

[alexcrichton]

Yeah you'll probably want to remove this try! and then match the result explicitly, checking the error and taking appropriate action depending on what it says

[pravic]

As I understand, there are two problems here: CTRL+Z and CTRL+C.

On CTRL+C ReadConsole returns TRUE with empty buffer and with last error ERROR_OPERATION_ABORTED (source). So, it can be handled easily here.

CTRL+Z on Windows works like CTRL+D on Unix, but Windows console functions doesn't handle it.

ReadFile (which can read from console too actually) has naive checking just for the first character of readed buffer about CTRLZ, but it doesnt know anything about line endings.

fgets treats CTRLZ as EOF when it found at start of line and this is default behavior for windows programs.

Thus, Rust must handle CTRLZ at some higher level, when processing the lines, I think.

Any thoughts?

[pravic]

cc @retep998

[retep998]

ReadFile suffers from doing the read in the current narrow code page which is almost never what you want (and especially does not work when using a multi byte encoding and it encounters a multi byte sequence).

Ctrl + C should simply invoke the current console ctrl handler, whatever it is, which by default terminates the process, although it can be configured via SetConsoleCtrlHandler.

I don't have any strong opinions on what we should do for Ctrl + Z.