diff --git a/ChangeLog b/ChangeLog index b1914f41b0..764ec81fbd 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +Sun Feb 3 16:54:15 2013 Nobuyoshi Nakada + + * ext/io/console/console.c (rawmode_opt): min is minimum characters, + not tenths. + Sun Feb 3 16:13:00 2013 Charlie Somerville * doc/security.rdoc: add first cut at a Ruby security document diff --git a/ext/io/console/console.c b/ext/io/console/console.c index cedc057bb3..61b31258e6 100644 --- a/ext/io/console/console.c +++ b/ext/io/console/console.c @@ -105,13 +105,12 @@ rawmode_opt(int argc, VALUE *argv, rawmode_arg_t *opts) if (!NIL_P(vopts)) { VALUE vmin = rb_hash_aref(vopts, ID2SYM(rb_intern("min"))); VALUE vtime = rb_hash_aref(vopts, ID2SYM(rb_intern("time"))); - VALUE v10 = INT2FIX(10); if (!NIL_P(vmin)) { - vmin = rb_funcall3(vmin, '*', 1, &v10); opts->vmin = NUM2INT(vmin); optp = opts; } if (!NIL_P(vtime)) { + VALUE v10 = INT2FIX(10); vtime = rb_funcall3(vtime, '*', 1, &v10); opts->vtime = NUM2INT(vtime); optp = opts; diff --git a/test/io/console/test_io_console.rb b/test/io/console/test_io_console.rb index 3acc4de5d5..2de15f1f24 100644 --- a/test/io/console/test_io_console.rb +++ b/test/io/console/test_io_console.rb @@ -25,6 +25,38 @@ class TestIO_Console < Test::Unit::TestCase } end + def test_raw_minchar + len = 0 + th = nil + helper {|m, s| + th = Thread.start { + m.print("a") + len += 1 + sleep 2 + m.print("1234567890") + len += 10 + } + assert_equal(["a", 1], [s.getch(min: 1), len]) + } + ensure + th.kill if th and th.alive? + end + + def test_raw_timeout + len = 0 + th = nil + helper {|m, s| + th = Thread.start { + sleep 2 + m.print("a") + len += 1 + } + assert_equal([nil, 0], [s.getch(time: 0.1), len]) + } + ensure + th.kill if th and th.alive? + end + def test_cooked helper {|m, s| assert_send([s, :echo?])